Skip to content
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

ARC: Add Authentication-Results: field (AR), if useful one is not found #1453

Merged
merged 6 commits into from
Nov 15, 2022

Conversation

ikedas
Copy link
Member

@ikedas ikedas commented Jul 24, 2022

See discussion on #1450.

ikedas added 3 commits July 24, 2022 16:22
  - arc_parameters & dkim_parameters use them each other as default.
  - arc_srvid uses signer_domain of ARC or DKIM as default.
  - Renamed subparameters for ARC.
@ikedas ikedas added enhancement ready A PR is waiting to be merged. Close to be solved labels Jul 24, 2022
@ikedas ikedas changed the title ARC: Add Authentication-Results: filed (AR), if useful one is not found ARC: Add Authentication-Results: field (AR), if useful one is not found Jul 24, 2022
@ikedas ikedas marked this pull request as ready for review July 27, 2022 04:23
@ikedas ikedas added this to the 6.2.72 milestone Sep 24, 2022
…(AR) about DKIM signature verification, add them, as done for ARC.

Conversely, if trusted AR(s) exist, now authentication information will be taken from them.
ikedas added a commit that referenced this pull request Nov 15, 2022
ARC: When ARC seal was added, DKIM signing should be forced (#1450)
Forwarded messages should also be ARC sealed if possible
ARC: Add Authentication-Results: field (AR), if useful one is not found
Following typo fix in Mail::DKIM::ARC::Signer
@ikedas ikedas merged commit 72c35ae into sympa-community:sympa-6.2 Nov 15, 2022
@ikedas ikedas deleted the add_ar_by_self branch November 15, 2022 07:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement ready A PR is waiting to be merged. Close to be solved
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant