validation on Yahoo raise authres.core.SyntaxError

Bug #1067070 reported by mardiros on 2012-10-15
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Authentication-Results-Python
Undecided
Unassigned

Bug Description

Yahoo mail Authentication-Results write the domainkeys result
always raise a SyntaxError due to domainkeys result.

>>> import authres
>>> arobj = authres.AuthenticationResultsHeader.parse('Authentication-Results: mta1040.mail.ukl.yahoo.com from=mail.mydomain.fr; domainkeys=neutral (no sig); from=mail.mydomain.fr; dkim=pass (ok)')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "build/bdist.linux-x86_64/egg/authres/__init__.py", line 191, in parse
  File "build/bdist.linux-x86_64/egg/authres/core.py", line 398, in parse
  File "build/bdist.linux-x86_64/egg/authres/core.py", line 410, in parse_value
  File "build/bdist.linux-x86_64/egg/authres/core.py", line 478, in _parse
authres.core.SyntaxError: Syntax error: Expected "none" or at least one resinfo at: from=mail.mydomain.fr; domainkeys=neutra...

mardiros (mardiros) on 2012-10-15
description: updated
Scott Kitterman (kitterman) wrote :

We claim domainkeys support (it's in the code). I'll look into this.

Scott Kitterman (kitterman) wrote :

Is this exactly what came from Yahoo? The header field you're showing is incorrect. What you have:

'Authentication-Results: mta1040.mail.ukl.yahoo.com from=mail.mydomain.fr; domainkeys=neutral (no sig); from=mail.mydomain.fr; dkim=pass (ok)'

is missing the ";" between the authserv ID (mta1040.mail.ukl.yahoo.com) and the start of the resinfo (from= ...), thus the error. If you change it to:

arobj = authres.AuthenticationResultsHeader.parse('Authentication-Results: mta1040.mail.ukl.yahoo.com; from=mail.mydomain.fr; domainkeys=neutral (no sig); from=mail.mydomain.fr; dkim=pass (ok)')

then it is processed fine.

Changed in authentication-results-python:
status: New → Incomplete
Launchpad Janitor (janitor) wrote :

[Expired for Authentication-Results-Python because there has been no activity for 60 days.]

Changed in authentication-results-python:
status: Incomplete → Expired
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers