Message 284293 - Python tracker

➜

This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author	bpoaugust
Recipients	barry, bpoaugust, r.david.murray
Date	2016-12-29.20:59:46
SpamBayes Score	-1.0
Marked as misclassified	Yes
Message-id	<1483045187.04.0.777957334246.issue29020@psf.upfronthosting.co.za>
In-reply-to

Content
I have just checked and AFAICT collapse_rfc2231_value is only called by get_filename and get_boundary in message.py. Both of these call get_param and default to unquote=True. So in all cases the parameter value passed to collapse_rfc2231_value will already have been unquoted. Calling unquote again is a bug. The obvious fix is to remove the calls to unquote from collapse_rfc2231_value. This will also fix issue28945.

I have just checked and AFAICT collapse_rfc2231_value is only called by get_filename and get_boundary in message.py.

Both of these call get_param and default to unquote=True.

So in all cases the parameter value passed to collapse_rfc2231_value will already have been unquoted. Calling unquote again is a bug.

The obvious fix is to remove the calls to unquote from collapse_rfc2231_value.

This will also fix issue28945.

History
Date	User	Action	Args
2016-12-29 20:59:47	bpoaugust	set	recipients: + bpoaugust, barry, r.david.murray
2016-12-29 20:59:47	bpoaugust	set	messageid: <1483045187.04.0.777957334246.issue29020@psf.upfronthosting.co.za>
2016-12-29 20:59:47	bpoaugust	link	issue29020 messages
2016-12-29 20:59:46	bpoaugust	create