Opened 2 years ago

Closed 2 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)

multiparser_json.diff​ (2.2 KB) - added by Facundo Batista 2 years ago.
Proof of concept patch showing what the fix would probably be

Download all attachments as: .zip

Change History (2)

Changed 2 years ago by Facundo Batista

Attachment: multiparser_json.diff​ added

Proof of concept patch showing what the fix would probably be

comment:1 Changed 2 years ago by Tim Graham

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.
Back to Top