Message232377
There is indeed a test failure that occurs without the patch. This is a new test I had added.
The reason is that in the existing implementation, when a boundary does not exist, the implementation does not include the trailing CRLF, LF or for that matter CR as part of the payload. I think that is not correct.
However, to keep this patch compatible with behavior of existing implementation I have updated the patch to strip a single CRLF, LR or CR from the payload if a boundary is not found. |
|
Date |
User |
Action |
Args |
2014-12-09 14:44:05 | rishi.maker.forum | set | recipients:
+ rishi.maker.forum, ishimoto, orsenthil, pitrou, teyc, r.david.murray, flox, BreamoreBoy, hynek, serhiy.storchaka, Chui.Tey |
2014-12-09 14:44:04 | rishi.maker.forum | set | messageid: <1418136244.52.0.862717016998.issue1610654@psf.upfronthosting.co.za> |
2014-12-09 14:44:04 | rishi.maker.forum | link | issue1610654 messages |
2014-12-09 14:44:04 | rishi.maker.forum | create | |
|