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

FEATURE: CAA_BUILDER: add issue_critical and issuewild_critical #2728

Merged
merged 2 commits into from Dec 18, 2023

Conversation

llange
Copy link
Contributor

@llange llange commented Dec 17, 2023

It is my understanding that - according to https://datatracker.ietf.org/doc/html/rfc8659 -
the critical flag can be applied to whatever tag of a CAA record.

This change of CAA_BUILDER adds the ability to provide this flag for the issue and
issuewild tags by setting respectively issue_critical and issuewild_critical to
true.

@cafferata
Copy link
Collaborator

Thank you @llange. See the GitBook preview deep link URL here:
https://docs.dnscontrol.org/~/revisions/fdpKWtbk1zQYZuTDVDh7/language-reference/domain-modifiers/caa_builder

Would you mind including issue_critical and issuewild_critical in the code example for completeness? 🙏

It is my understanding that - according to https://datatracker.ietf.org/doc/html/rfc8659 -
the critical flag can be applied to whatever tag of a CAA record.

This change of CAA_BUILDER adds the ability to provide this flag for the `issue` and
`issuewild` tags by setting respectively `issue_critical` and `issuewild_critical` to
`true`.
@llange
Copy link
Contributor Author

llange commented Dec 18, 2023

Hi @cafferata, thanks for your suggestion. I thus updated the doc and tweaked it a little:

Let me know if it's too much or OK.

Copy link
Collaborator

@cafferata cafferata left a comment

Choose a reason for hiding this comment

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

Thank you @llange. I think the new code examples definitely add value. Thanks for that. 🤙

@cafferata
Copy link
Collaborator

PS. @tlimoncelli will also do a review and take care of the merge. 🚀

@tlimoncelli tlimoncelli merged commit edf0471 into StackExchange:master Dec 18, 2023
1 check passed
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.

None yet

3 participants