Opened 7 weeks ago

Closed 7 weeks 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 7 weeks ago.
Proof of concept patch showing what the fix would probably be

Download all attachments as: .zip

Change History (2)

Changed 7 weeks ago by Facundo Batista

Attachment: multiparser_json.diff added

Proof of concept patch showing what the fix would probably be

comment:1 Changed 7 weeks ago by Tim Graham

Component: UncategorizedHTTP handling
Resolution: duplicate
Status: newclosed
Type: UncategorizedCleanup/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