trusteddomainproject / OpenARC

Open source ARC implementation
BSD 2-Clause "Simplified" License
132 stars 46 forks source link

OpenARC is not merging AR headers #77

Closed geneshuman closed 6 years ago

geneshuman commented 6 years ago

The following 2 tests of the signing ARC test suite fail:

ar_merged1 ar_merged2

The specification states that upon generation of AAR headers in the ARC set, the intermediary is to merge all AR headers from trusted intermediaries into one.

AntiFreeze commented 6 years ago

It is merging, but it's broken. Output I just got from the latest:

0150623.gappssmtp.com header.i=@sethblank-com.20150623.gappssmtp.com header.b=h6
0zF85p reason="verification failed; unprotected key"; dkim-adsp=unknown (unprote
cted policy) reason="verification failed; unprotected key"; dkim-atps=neutral re
ason="verification failed; unprotected key"dmarc=none (p=none dis=none) header.f
rom=sethblank.comarc=none

Obviously that final statement header.from=sethblank.comarc=none" should be header.f rom=sethblank.com; arc=none

AntiFreeze commented 6 years ago

I can no longer reproduce this issue with the latest code from develop.