DKIM2 docs: reply-code guidance is §10.4 not §11.1; any-one-passes ve… - #863
Merged
Conversation
…rdict is Momentum policy, not §11.6
✅ Deploy Preview for support-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
dkoerichbird
approved these changes
Jul 28, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
…rdict is Momentum policy, not §11.6
What Changed
How To Test or Verify
PR Checklist
Below are some checklists to follow for the correct procedure in different circumstance. The first list ("All PRs Checklist") should be followed for ALL PRs. The next 2 are additive to this list depending on what type of PR you are using.
For example: If you are submitting a content change to one of the support documents, your checklist would include the:
If you are submitting a feature addition, enhancement, or bug fix, your checklist would include the:
All PRs Checklist
team-FEorteam-SAZ)Content Changes Checklist
examples/article.mdin the root of the project directory and on the momentum doc's preface articleDevelopment Changes Checklist (some checks are automatic github actions and will not be listed here. ie. "all tests pass")
cypress/directory in the root of the projectNote
Low Risk
Documentation-only corrections to spec section references and product-behavior clarity; no runtime or API changes.
Overview
DKIM2 verification docs retarget SMTP reply guidance from §11.1 to §10.4: section title, table headers, inline references, anchor link (
#smtp-response-codes-104-guidance), and the example hook comments. Thepermerrorrow no longer claims MUST NOT 4xx in the suggested-action column; key rules now state onlyfail(crypto failure) MUST NOT use 4xx, while other permanent errors (permerror) SHOULD use 5xx.DKIM2 signing docs clarify that when multiple sig-sets are present, “overall verdict passes if any one validates” is Momentum’s behavior (not presented as generic §11.6 wording), with minor line wrapping on the combined
DKIM2-Signaturedescription.Both pages bump
lastUpdatedto 07/27/2026.Reviewed by Cursor Bugbot for commit 8a5a7e7. Bugbot is set up for automated code reviews on this repo. Configure here.