Opened 5 years ago
Closed 5 years ago
#28678 closed Cleanup/optimization (duplicate)
Multipart parser should decode those JSON parts that are flagged as such
Reported by: | Facundo Batista | Owned by: | nobody |
---|---|---|---|
Component: | HTTP handling | Version: | 1.11 |
Severity: | Normal | Keywords: | |
Cc: | Triage Stage: | Unreviewed | |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
If a part of the multipart body has a proper JSON mimetype, it should be automatically decoded.
Attached is a proof of concept patch (no tests so far, and probably is there a better way to check the mimetype).
Attachments (1)
Change History (2)
Changed 5 years ago by
Attachment: | multiparser_json.diff added |
---|
comment:1 Changed 5 years ago by
Component: | Uncategorized → HTTP handling |
---|---|
Resolution: | → duplicate |
Status: | new → closed |
Type: | Uncategorized → Cleanup/optimization |
I think we couldn't make this change as proposed because it could be backwards incompatible (data could now be a different type in request.POST
, correct?).
I think we can mark this as a duplicate of #21442.
Note: See
TracTickets for help on using
tickets.
Proof of concept patch showing what the fix would probably be