Repository navigation
email.quoprimime.decode() leaves stray \r when eol='\r\n' #145831
Description
Bug report
Bug description:
Issue
email.quoprimime.decode() strips only one character when removing a trailing line ending that wasn't in the original input. When eol='\r\n' (2 characters), only the \n is removed, leaving a stray \r.
The bug is in this line:
decoded = decoded[:-1]It should be decoded[:-len(eol)].
Reproducer
import email.quoprimime as qp
print(repr(qp.decode('abc', eol='\r\n'))) # 'abc\r' — expected 'abc'
print(repr(qp.decode('a\nb', eol='\r\n'))) # 'a\r\nb\r' — trailing \r is spuriousCPython versions tested on:
CPython main branch
Operating systems tested on:
No response
Activity
khalidsaidi commentedon Mar 25, 2026
gh-145831: email.quoprimime: `decode()` leaves stray `\r` when `eol='…
pythongh-145831: email.quoprimime: `decode()` leaves stray `\r` when …
pythongh-145831: email.quoprimime: `decode()` leaves stray `\r` when …
[3.14] gh-145831: email.quoprimime: `decode()` leaves stray `\r` when…
[3.13] gh-145831: email.quoprimime: `decode()` leaves stray `\r` when…
bitdancer commentedon Apr 10, 2026
Thanks, Stephan.
pythongh-145831: email.quoprimime: `decode()` leaves stray `\r` when …