-
Notifications
You must be signed in to change notification settings - Fork 45
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.
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
Remove t from the list of required AS tags #121
Conversation
"t" is imported from RFC 6376 section 3.5, which states: t= Signature Timestamp (plain-text unsigned decimal integer; RECOMMENDED, default is an unknown creation time). The time that this signature was created. Its inclusion in this list results in spurious ARC verification failures.
@martinbogo It's been almost ten months since #123 (comment); any progress on merging this critical bugfix? |
We are working on getting OpenDMARC 1.4.1 out the door. Once that's
completed, we'll circle back to working on OpenARC and OpenDKIM.
…On Sat, Apr 10, 2021 at 8:51 PM flowerysong ***@***.***> wrote:
@martinbogo <https://github.com/martinbogo> It's been almost ten months
since #123 (comment)
<#123 (comment)>;
any progress on merging this critical bugfix?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#121 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAB5KKNX2HL7QG2U55ZI633TID6CFANCNFSM4ISGVJVA>
.
|
@martinbogo That's really not acceptable. This is a bug that makes the library unfit for purpose (since it breaks validation of ARC signatures from any domain hosted by Microsoft) and the fix is a one-line change. If you want to pretend this project is viable, you need to make more progress in ten months than "we'll circle back to it". |
@*flowerysong*
We are three people with limited time, and *extremely* limited resources.
The only way anything gets done is by focusing on one thing at a time,
putting the work in, and then moving to the next item. Your feedback is
valid, its true, and yet ultimately not that helpful. Sorry to be blunt,
but being told we aren't working hard enough, or long enough, or paying
attention to a particular branch or fix won't make it happen. OTOH, making
it easier for us by making a codefix, PR, testing the merge in a fork and
then reporting back? That is helpful, especially if it's against the
current development branch
The working group behind ARC seals and DKIM is (slowing) working through to
a new RFC and milestones. I recently recruited @thegushi ( Dan ) to help
so we can get merge in critical bugfixes in DMARC and move 1.4.1 out the
door ( which is expected to drop in the next 72 hours ).
After that, Murray, Dan, and I will refocus and look at critical and
community CVE and serious bugfixes in DKIM and ARC, especially as they
relate to recent issues solved by working on DMARC.
Keep submitting PR's and issues.
We'll keep on working.
…-Martin
On Fri, Apr 16, 2021 at 1:57 PM flowerysong ***@***.***> wrote:
@martinbogo <https://github.com/martinbogo> That's really not acceptable.
This is a bug that makes the library unfit for purpose (since it breaks
validation of ARC signatures from any domain hosted by Microsoft) and the
fix is a one-line change.
If you want to pretend this project is viable, you need to make more
progress in ten months than "we'll circle back to it".
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#121 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAB5KKLU6AIF7DTBWLS4JVDTJCCBDANCNFSM4ISGVJVA>
.
|
"t" is imported from RFC 6376 section 3.5, which states: t= Signature Timestamp (plain-text unsigned decimal integer; RECOMMENDED, default is an unknown creation time). The time that this signature was created. Its inclusion in this list results in spurious ARC verification failures. trusteddomainproject#121
"t" is imported from RFC 6376 section 3.5, which states:
Its inclusion in this list results in spurious ARC verification failures.