dkimpy

Merge ~schopin/dkimpy:py3.12 into dkimpy:master

  1. Git
  2. lp:~schopin/dkimpy
  3. py3.12
  4. Merge into master
Proposed by Simon Chopin
Status: Merged
Approved by: Scott Kitterman
Approved revision: 073a0a1169fd99f490f2c67a09df60c7027ed505
Merge reported by: Scott Kitterman
Merged at revision: 073a0a1169fd99f490f2c67a09df60c7027ed505
Proposed branch: ~schopin/dkimpy:py3.12
Merge into: dkimpy:master
Diff against target: 13 lines (+1/-1)
1 file modified
dkim/__init__.py (+1/-1)
Reviewer Review Type Date Requested Status
Scott Kitterman Approve
Review via email: mp+461983@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Scott Kitterman (kitterman) wrote :

Looks good. Thanks.

review: Approve
Revision history for this message
Scott Kitterman (kitterman) wrote :

Merged into master.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/dkim/__init__.py b/dkim/__init__.py
2index 8b43f15..62971d0 100644
3--- a/dkim/__init__.py
4+++ b/dkim/__init__.py
5@@ -302,7 +302,7 @@ def validate_signature_fields(sig, mandatory_fields=[b'v', b'a', b'b', b'bh', b'
6 try:
7 str(sig[b'd'], 'ascii')
8 # No specials, which is close enough
9- if re.findall(b"[\(\)<>\[\]:;@\\,]", sig[b'd']):
10+ if re.findall(rb"[\(\)<>\[\]:;@\\,]", sig[b'd']):
11 raise ValidationError("d= value is not valid (%s)" % sig[b'd'])
12 except UnicodeDecodeError as e:
13 # Not an ASCII domain

Subscribers

People subscribed via source and target branches

to all changes: