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

Lars Eggert COMMENT (3) #19

Open
vixie opened this issue Jan 7, 2024 · 5 comments
Open

Lars Eggert COMMENT (3) #19

vixie opened this issue Jan 7, 2024 · 5 comments

Comments

@vixie
Copy link
Contributor

vixie commented Jan 7, 2024

Section 3.1, paragraph 3

     At the time of writing, most DNS server software did not set the DF
     bit for IPv4, and many OS kernel constraints make it difficult to set
     the DF bit in all cases.  Best Current Practice documents should not
     specify what is currently impossible, so R2, which is setting the DF
     bit, is "MAY" rather than "SHOULD".

Maybe I'm familiar with different kernels, but all the ones I am
familiar with (except for some IoT platforms) readily offer socket
options to set DF (and prevent stack fragmentation in v6).

@paulwouters
Copy link

this was rewritten based on various ballot comments. My propsed PR contains:

  and many OS kernel constraints make it difficult to set the DF bit in all cases.
  Best Current Practice documents should not specify what is currently impossible,
  so R2, which is setting the DF bit, is "MAY" rather than "SHOULD".

@vixie
Copy link
Contributor Author

vixie commented Feb 18, 2024 via email

@paulwouters
Copy link

You are correct. I thought this has been rewritten. I think it should say:

Best Current Practice documents should not specify what is currently impossible,
  so R2, which is setting the DF bit, is "SHOULD" rather than "MUST"

@vixie
Copy link
Contributor Author

vixie commented Feb 19, 2024 via email

@moonshiner
Copy link

Agree that "if supported, must" was the WG consensus and we are feel to push back on the IESG on this point.

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

No branches or pull requests

3 participants