Mention draft-ietf-dprive-padding-policy which provides guidelines #207
Conversation
This file contains 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
on padding length which was absent from RFC 7830.
mcmanus
reviewed
Jul 25, 2018
| @@ -631,7 +631,8 @@ DNS queries. | |||
| HTTP/2 provides further advice about the use of compression | |||
| ({{RFC7540}} Section 10.6) and padding ({{RFC7540}} Section 10.7 ). | |||
| DoH Servers can also add DNS padding {{RFC7830}} if the DoH client requests | |||
| it in the DNS query. | |||
| it in the DNS query. Additionally, guidelines for choosing the padding length | |||
| can be found in {{draft-ietf-dprive-padding-policy}}. | |||
There was a problem hiding this comment.
that's a reference to experimental phrased as if its standards track guidance...
There was a problem hiding this comment.
I can modify it, and re-send a pull request. dprive-padding-policy has been implemented in open source software such as pydig & knot.
There was a problem hiding this comment.
Agreed. I would perhaps say "initial attempts to offer guidance on choosing the padding length..."
There was a problem hiding this comment.
Thanks @moonshiner for the suggestion.
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.
on padding length which was absent from RFC 7830.