Skip to content

docs: Add info regarding DKIM key rotation and non-expiry#4076

Merged
polarathene merged 6 commits intomasterfrom
docs/dkim-key-rotation-expiry
Jun 19, 2024
Merged

docs: Add info regarding DKIM key rotation and non-expiry#4076
polarathene merged 6 commits intomasterfrom
docs/dkim-key-rotation-expiry

Conversation

@polarathene
Copy link
Copy Markdown
Member

Description

A recent discussion about this topic suggested including a mention about this in our docs.

I'm not 100% sure I'm correct about this advice, so if you know any better please correct me 👍

Copy link
Copy Markdown
Member

@georglauterbach georglauterbach left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I really like the whole update; DKIM key rotation is really useful. There are two minor tweaks I can recommend down below.

Comment thread docs/content/config/best-practices/dkim_dmarc_spf.md Outdated
Comment thread docs/content/config/best-practices/dkim_dmarc_spf.md Outdated
Co-authored-by: Georg Lauterbach <[email protected]>
Comment thread docs/content/config/best-practices/dkim_dmarc_spf.md Outdated
Co-authored-by: Brennan Kinney <[email protected]>
Comment thread docs/content/config/best-practices/dkim_dmarc_spf.md Outdated
casperklein
casperklein previously approved these changes Jun 18, 2024
@polarathene polarathene dismissed stale reviews from casperklein and georglauterbach via af85a3f June 19, 2024 00:16
@polarathene polarathene enabled auto-merge (squash) June 19, 2024 08:38
@github-actions
Copy link
Copy Markdown
Contributor

Documentation preview for this PR is ready! 🎉

Built with commit: d00ea4d

@polarathene polarathene merged commit 8a082be into master Jun 19, 2024
@polarathene polarathene deleted the docs/dkim-key-rotation-expiry branch June 19, 2024 08:39
@georglauterbach georglauterbach modified the milestones: v14.1.0, v15.0.0 Sep 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants