Re: [Django] #31293: MultiPartParser support double quotes

[Django]

#31293: MultiPartParser support double quotes
-+-
 Reporter:  007  |Owner:  007
 Type:   |   Status:  closed
  Cleanup/optimization   |
Component:  HTTP handling|  Version:  master
 Severity:  Normal   |   Resolution:  fixed
 Keywords:   | Triage Stage:  Ready for
 |  checkin
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by Mariusz Felisiak ):

 * status:  assigned => closed
 * resolution:   => fixed


Comment:

 In [changeset:"e65fea9292ffdeb9bb76062f6cb2a5ff514ae969" e65fea92]:
 {{{
 #!CommitTicketReference repository=""
 revision="e65fea9292ffdeb9bb76062f6cb2a5ff514ae969"
 Fixed #31293 -- Allowed MultiPartParser to handle double-quoted encoded
 headers.
 }}}

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/064.057c6278eb6d724de3aba273d036c8e2%40djangoproject.com.

Re: [Django] #31293: MultiPartParser support double quotes

[Django]

#31293: MultiPartParser support double quotes
-+-
 Reporter:  007  |Owner:  007
 Type:   |   Status:  assigned
  Cleanup/optimization   |
Component:  HTTP handling|  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Ready for
 |  checkin
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by Carlton Gibson):

 * stage:  Accepted => Ready for checkin


-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/064.2b1b14a5cb03552280a03e13edb4d3af%40djangoproject.com.

Re: [Django] #31293: MultiPartParser support double quotes

[Django]

#31293: MultiPartParser support double quotes
--+
 Reporter:  007   |Owner:  007
 Type:  Cleanup/optimization  |   Status:  assigned
Component:  HTTP handling |  Version:  master
 Severity:  Normal|   Resolution:
 Keywords:| Triage Stage:  Accepted
Has patch:  1 |  Needs documentation:  0
  Needs tests:  0 |  Patch needs improvement:  0
Easy pickings:  0 |UI/UX:  0
--+
Changes (by Carlton Gibson):

 * stage:  Ready for checkin => Accepted


Comment:

 GitHub is momentarily dying on me. I have small edits to push. I will try
 again later.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/064.28f2259fbf8962d9196ab37e5e270bc4%40djangoproject.com.

Re: [Django] #31293: MultiPartParser support double quotes

[Django]

#31293: MultiPartParser support double quotes
-+-
 Reporter:  007  |Owner:  007
 Type:   |   Status:  assigned
  Cleanup/optimization   |
Component:  HTTP handling|  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Ready for
 |  checkin
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by Carlton Gibson):

 * stage:  Accepted => Ready for checkin


Comment:

 PR looks correct. Thank you.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/064.096fae2d45f009638b17c6a451bf6093%40djangoproject.com.

Re: [Django] #31293: MultiPartParser support double quotes

[Django]

#31293: MultiPartParser support double quotes
--+
 Reporter:  007   |Owner:  007
 Type:  Cleanup/optimization  |   Status:  assigned
Component:  HTTP handling |  Version:  master
 Severity:  Normal|   Resolution:
 Keywords:| Triage Stage:  Accepted
Has patch:  1 |  Needs documentation:  0
  Needs tests:  0 |  Patch needs improvement:  0
Easy pickings:  0 |UI/UX:  0
--+
Changes (by Carlton Gibson):

 * stage:  Unreviewed => Accepted


Comment:

 Thanks for the report. First glance at the PR looks reasonable.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/064.5a850d1b94870323db7709bc41dac5c0%40djangoproject.com.

Re: [Django] #31293: MultiPartParser support double quotes

[Django]

#31293: MultiPartParser support double quotes
-+-
 Reporter:  007  |Owner:  007
 Type:   |   Status:  assigned
  Cleanup/optimization   |
Component:  HTTP handling|  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:
 |  Unreviewed
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by 007):

 * has_patch:  0 => 1


-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/064.d585773b07f65dea1e4a61599b6e2c9a%40djangoproject.com.

Re: [Django] #31293: MultiPartParser support double quotes

[Django]

#31293: MultiPartParser support double quotes
-+-
 Reporter:  007  |Owner:  007
 Type:   |   Status:  assigned
  Cleanup/optimization   |
Component:  HTTP handling|  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:
 |  Unreviewed
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by 007):

 * owner:  nobody => 007
 * status:  new => assigned


-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/064.16a3579416ce0c568658878e4620a9d8%40djangoproject.com.

[Django] #31293: MultiPartParser support double quotes

[Django]

#31293: MultiPartParser support double quotes
+
   Reporter:  007   |  Owner:  nobody
   Type:  Cleanup/optimization  | Status:  new
  Component:  HTTP handling |Version:  master
   Severity:  Normal|   Keywords:
   Triage Stage:  Unreviewed|  Has patch:  0
Needs documentation:  0 |Needs tests:  0
Patch needs improvement:  0 |  Easy pickings:  0
  UI/UX:  0 |
+
 Although the rfc2231 document does not indicate that values can be wrapped
 in double quotes. However, some third-party tools wrap the value in double
 quotation marks when wrapping HTTP requests (such as the filename of the
 file uploaded by PostmanCanary). This results in double quotes for the
 filename at the end of Django parsing.

 Postman request body:
 {{{
 677822685948106391633425
 Content-Disposition: form-data; name="file"; filename="测试.txt";
 filename*="UTF-8''%E6%B5%8B%E8%AF%95.txt"
 Content-Type: text/plain

 test
 677822685948106391633425--
 }}}

 django got filename is `测试.txt"` not `测试.txt`

 This is not a bug of Django. But I suggest Django do some compatibility
 processing.

 CPython is also compatible with this situation.
 
[https://hg.python.org/cpython/file/default/Lib/test/test_email/test_headerregistry.py]




 [https://github.com/django/django/pull/12485]

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/049.3ad524c8b361a77897d08813529726aa%40djangoproject.com.