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

Lots of editorial fixes #2

Merged
merged 1 commit into from Feb 28, 2018

Conversation

Projects
None yet
2 participants
@paulehoffman
Copy link
Contributor

commented Feb 28, 2018

No description provided.

@wkumari
Copy link
Collaborator

left a comment

Thank you very much - these improve the document, and sending it as a PR is much appreciated.

@@ -492,7 +497,7 @@
Sivaraman, Petr Spacek. Andrew Sullivan, Paul Vixie, Duane Wessels and
Paul Wouters for their helpful feedback.</t>

<t>[TODO: Add people who have contributed!]</t>
<t>[TODO: Add more people who have contributed!]</t>

This comment has been minimized.

Copy link
@wkumari

wkumari Feb 28, 2018

Collaborator

:-P

The "kskroll-sentinel-not-ta-&lt;tag-index&gt;" label is used in
a query where the response can answer whether this the key tag
of a trust anchor that is NOT in currently trusting.</t>

This comment has been minimized.

Copy link
@wkumari

wkumari Feb 28, 2018

Collaborator

The last sentence / phrase is hard to parse (" that is NOT in currently trusting."); I'll patch it later, no need up update PR.

@wkumari wkumari merged commit 967ff9a into APNIC-Labs:master Feb 28, 2018

wkumari added a commit that referenced this pull request Feb 28, 2018

From -02 to -03:
Integrated / published comments from Paul in GitHub PR #2 - #2
Made the keytab be decimal, not hex (thread / consensus in https://mailarchive.ietf.org/arch/msg/dnsop/Kg7AtDhFRNw31He8n0_bMr9hBuE )

@paulehoffman paulehoffman deleted the paulehoffman:editorial-peh branch Mar 1, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.