Skip to content

gh-137855: email.quoprimime removing re import #132046

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
Marius-Juston wants to merge 11 commits into from
+27 −10
Closed

gh-137855: email.quoprimime removing re import #132046

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
c670b11
Removed re hex
Marius-Juston Apr 3, 2025
08cdc0b
implace replace, removed valid_hex parameter
Marius-Juston Apr 3, 2025
a3ef550
joined to big if statement
Marius-Juston Apr 3, 2025
22e6d9e
added news
Marius-Juston Apr 3, 2025
232bb55
inline character assigment
Marius-Juston Apr 3, 2025
3ada67a
use cache for hex to char + instead of single character append use sl…
Marius-Juston Apr 3, 2025
9e3cc1f
inplace assignment with walrus
Marius-Juston Apr 3, 2025
3287564
removed news since should probably be "skip news" tagged
Marius-Juston Apr 3, 2025
8362a2e
fast pass for no '='
Marius-Juston Apr 3, 2025
81ae23a
Update Lib/email/quoprimime.py
Marius-Juston Apr 4, 2025
8659486
faster string concatenation
Marius-Juston Apr 4, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
implace replace, removed valid_hex parameter
  • Loading branch information
@Marius-Juston
Marius-Juston committed Apr 3, 2025
commit 08cdc0b97a4354a5b0b4afc2380c9fcc3d70d76b
15 changes: 5 additions & 10 deletions Lib/email/quoprimime.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -286,24 +286,19 @@ def header_decode(s):
quoted-printable (like =?iso-8859-1?q?Hello_World?=) -- please use
the high level email.header class for that functionality.
"""
s = s.replace('_', ' ')

valid_hex = '0123456789ABCDEFabcdef'

i = 0
# Check for regex =[a-fA-F0-9]{2}

result = []

while i < len(s):
if s[i] == '=' and i + 2 < len(s):
max_s_check_len = s - 2
for i, c in enumerate(s):
if c == '=' and i < max_s_check_len:
hex_part = s[i: i + 3]

if (hex_part[1] in valid_hex) and (hex_part[2] in valid_hex):
if (hex_part[1] in hexdigits) and (hex_part[2] in hexdigits):
result.append(unquote(hex_part))
i += 3
continue
result.append(s[i])
result.append(' ' if c == '_' else c)
i += 1

return ''.join(result)