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

Add files via upload #24

Merged
merged 5 commits into from
Mar 6, 2024
Merged

Add files via upload #24

merged 5 commits into from
Mar 6, 2024

Conversation

tireddy2
Copy link
Collaborator

@tireddy2 tireddy2 commented Mar 6, 2024

No description provided.

@tireddy2 tireddy2 requested a review from boucadair as a code owner March 6, 2024 05:42
draft-ietf-add-resolver-info.md Outdated Show resolved Hide resolved
draft-ietf-add-resolver-info.md Outdated Show resolved Hide resolved
@@ -109,9 +105,9 @@ Reputation:
using the RESINFO RR type and QNAME of "resolver.arpa". In this case, a client has to contend
with the risk that a resolver does not support RESINFO. The resolver might
pass the query upstream, and then the client can receive a positive RESINFO response either
from a legitimate DNS resolver or an attacker. The DNS client MUST set the Recursion
Desired (RD) bit of the query to 0 to ensure that the response is provided by the resolver.
Copy link
Owner

Choose a reason for hiding this comment

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

Shoudn't we keep this part on RD?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

It is discussed in the WG that setting the RD flag will not solve the problem, see https://mailarchive.ietf.org/arch/msg/add/jZUYZrgqcmTXa6xZlYqNOZS-7wA/

@boucadair boucadair merged commit a8cb649 into main Mar 6, 2024
3 checks passed
@boucadair boucadair deleted the tireddy2-patch-1 branch March 6, 2024 07:05
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