[issue18324] set_payload does not handle binary payloads correctly

R. David Murray report at bugs.python.org
Thu Aug 22 03:18:56 CEST 2013


R. David Murray added the comment:

Thanks, Vajrasky. The v2 patch was almost correct.  What you couldn't know without being as deeply enmeshed in this code as I am is that the test failures from the encoders module were actually invalid.  We'd previously "fixed" them, but the fixes were incorrectly compensating for this bug in set_payload.  Once this bug was fixed, those "fixes" just needed to be backed out, a 'decode=True' added to their get_payload calls, and all the tests pass.

I *think* this is the last inconsistency in the model.  I hope.

----------
resolution:  -> fixed
stage: test needed -> committed/rejected
status: open -> closed

_______________________________________
Python tracker <report at bugs.python.org>
<http://bugs.python.org/issue18324>
_______________________________________


More information about the Python-bugs-list mailing list