Automatically switch to verifying if no ARH is found

[alevesely]

If there is no ARH, but an (unverified) DKIM-Signature exists, no verification is carried out. The following error sometimes is displayed, presumably due to an empty ARH.

2016-01-17 19:50:03 DKIM_Verifier.AuthVerifier ERROR Error: Parsing error (resource://dkim_verifier/ARHParser.jsm:249) JS Stack trace: [email protected]:249:1 < [email protected]:186:14 < [email protected]:161:21 < [email protected]:161:10 < _authVerifier_verify/promise<@AuthVerifier.jsm:103:17

[lieser]

I can not confirm that with a non existing ARH no verification is carried out. Are you perhaps instead meaning an ARH header without a DKIM result in it? For this the verification is indeed not carried out. This behavior will be corrected.

About the parsing error:
Are you meaning with an empty ARH header something like Authentication-Results: or Authentication-Results: authserv-id;? This are not correct ARH headers. The correct way to indicate that no message authentication was done is Authentication-Results: authserv-id; none. Although this currently also produces an error (which will be corrected), it is not a paring error.
If you still think the ARH header which produces the parsing error is correct, please post it (or send it to [email protected]) so I can look into it in detail.

[alevesely]

There is a silly:
Authentication-Results: tana.it; auth=pass (details omitted)
and there is a DKIM-Signature. It looks as if parsing throws an exception, so nothing else is going to be done. The status bar says "DKIM: No Signature", although there is one.

[lieser]

Could you please send me the complete header that is giving the parsing error to [email protected]?

[lieser]

The verification should now continue if the ARH header does not include a DKIM result. You can download a pre-release form https://github.com/lieser/dkim_verifier/releases.