Skip to content

Conversation

aidecoe
Copy link
Contributor

@aidecoe aidecoe commented Dec 2, 2017

DMARC reports usually come in ZIP files. To check the report you have to
unpack and search thru XML document which is very tedious. The filter
tags the message as follows:

  • if there's any SPF failure in any attachment, tag the message with
    "dmarc-spf-fail" tag, otherwise tag with "dmarc-spf-ok"
  • if there's any DKIM failure in any attachment, tag the message with
    "dmarc-dkim-fail" tag, otherwise tag with "dmarc-dkim-ok"

DMARC reports usually come in ZIP files. To check the report you have to
unpack and search thru XML document which is very tedious. The filter
tags the message as follows:

- if there's any SPF failure in any attachment, tag the message with
"dmarc-spf-fail" tag, otherwise tag with "dmarc-spf-ok"
- if there's any DKIM failure in any attachment, tag the message with
"dmarc-dkim-fail" tag, otherwise tag with "dmarc-dkim-ok"
@GuillaumeSeren
Copy link
Collaborator

Hey @aidecoe thank you for the patch
This looks good to me and it is a external filter, so let's merge it.

@GuillaumeSeren GuillaumeSeren merged commit ff01897 into afewmail:master Dec 3, 2017
@flokli
Copy link
Member

flokli commented Dec 3, 2017

@aidecoe Same as in #170: Could you please add an example mail for each of the 4 different tags, so they can be intergrated in the test harness?

@andir
Copy link
Member

andir commented Dec 3, 2017

Why I am the author of that commit? That looks wrong o.O

@GuillaumeSeren
Copy link
Collaborator

GuillaumeSeren commented Dec 3, 2017

Be careful @andir it's a trap we will ask you to come back on the project ! 🍺

@GuillaumeSeren
Copy link
Collaborator

But I think you are just the first maintainer to validate the review so GH take the first one

@aidecoe
Copy link
Contributor Author

aidecoe commented Dec 4, 2017

@flokli I see @andir has taken all the credit for my DMARC filter so now he provides example e-mail messages.

andir pushed a commit that referenced this pull request Dec 4, 2017
Inspect DMARC reports and tag the message
@andir
Copy link
Member

andir commented Dec 5, 2017

@aidecoe I removed myself from the Author field which I never added...

@GuillaumeSeren
Copy link
Collaborator

@andir Yes it is automatically added when you review so, joke aside not a big deal,
but I hope this will not prevent you to review patches again in the future !

You can reply instead of approving if don't want your name in the commit 🍺 😄 🍻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants