Repository navigation
gh-128949: Preserve payload line endings in BytesParser.parse - #157726
sankalpsthakur wants to merge 4 commits into
Conversation
|
Found a mailbox compatibility regression while investigating the Windows failures on 5a0df99: with CRLF storage, Maildir/MH now return CRLF payloads while A candidate follow-up keeps universal-newline conversion in |
|
The mailbox fix is in |
Link the documented class rather than its inherited parse method. The author reviewed and approved this one-line correction. AI-assisted: Codex.
|
The NEWS reference correction is now in |
db3d2c1 to
d19b94e
Compare
Fixes #128949. Thanks to tnakamot and medmunds for the reports.
Preserve CRLF and bare CR bytes when parsing binary files with
BytesParserandBytesHeaderParser, so unencoded attachments agree withparsebytes(). Header recognition and ownership of the caller's stream are unchanged.The mailbox follow-up in
9634201retainsmailbox.Message's existing universal-newline behaviour for binary streams.004a504corrects the NEWS link to the documentedBytesHeaderParserclass without changing runtime code.Regression coverage includes both policies, payload/transfer-encoding variants, multipart attachments, an 8192-byte read boundary, header-only parsing and mailbox construction from streams/bytes.
All reported upstream checks pass on
004a504, including Docs, Doctest, EPUB, Windows, macOS, Linux and the required-check aggregate. Maintainer review remains. The NEWS correction also removed exactly the failing reference in local nitpicky Sphinx output; the repository's new-NEWS warning gate changed from exit 255 to 0, with no new warnings.AI tools assisted with implementation, validation and this description. The author reviewed and approved the latest correction.