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

Change how the validation label is constructed #25

Merged
merged 5 commits into from
Feb 10, 2024
Merged

Change how the validation label is constructed #25

merged 5 commits into from
Feb 10, 2024

Conversation

aaomidi
Copy link
Owner

@aaomidi aaomidi commented Sep 1, 2023

Addresses #13

Trying to get closer to the generic dns validations method document: https://datatracker.ietf.org/doc/draft-ietf-dnsop-domain-verification-techniques/

@aaomidi aaomidi force-pushed the dnsops branch 6 times, most recently from c31260f to 2916810 Compare September 1, 2023 19:20
Repository owner deleted a comment from netlify bot Sep 1, 2023
@github-actions
Copy link

github-actions bot commented Sep 1, 2023

@github-actions github-actions bot temporarily deployed to pull request September 1, 2023 19:54 Inactive
@github-actions github-actions bot temporarily deployed to pull request September 1, 2023 19:57 Inactive
@github-actions github-actions bot temporarily deployed to pull request September 1, 2023 19:57 Inactive
@github-actions github-actions bot temporarily deployed to pull request September 1, 2023 20:17 Inactive
@github-actions github-actions bot temporarily deployed to pull request September 4, 2023 18:13 Inactive
Copy link
Collaborator

@daknob daknob left a comment

Choose a reason for hiding this comment

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

In addition to these suggestions, the Security Considerations are no longer true:

As this challenge that is introduced only differs in the left-most label of the domain name from the existing dns-01 challenge, the same security considerations apply.

If we move forward with this, we'd need to redo our security assessment and impact analysis, and see if anything noteworthy comes up.

draft-ietf-acme-dns-account-01.mkd Outdated Show resolved Hide resolved
draft-ietf-acme-dns-account-01.mkd Show resolved Hide resolved
draft-ietf-acme-dns-account-01.mkd Outdated Show resolved Hide resolved
@github-actions github-actions bot temporarily deployed to pull request September 7, 2023 21:30 Inactive
@aaomidi aaomidi requested a review from daknob September 7, 2023 21:39
@daknob
Copy link
Collaborator

daknob commented Sep 7, 2023

Seems okay so far, pending the changes from the dnsop draft. We'd like to cite that somewhere.

@aaomidi aaomidi merged commit 19e266f into main Feb 10, 2024
2 checks 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.

2 participants