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
net: use the extended RCode from EDNS(0) OPT resources #61695
Conversation
Change-Id: I8459d87c88a6d16e88aa44f4e68b6e14bf74d83c
This PR (HEAD: 8dcbf68) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/514835. Important tips:
|
Message from Mateusz Poliwczak: Patch Set 1: Run-TryBot+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/514835. |
c1f660e
to
958d076
Compare
Message from Gopher Robot: Patch Set 1: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/514835. |
This PR (HEAD: 958d076) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/514835. Important tips:
|
Message from Mateusz Poliwczak: Patch Set 2: Run-TryBot+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/514835. |
Message from Gopher Robot: Patch Set 2: (2 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/514835. |
Message from Gopher Robot: Patch Set 2: TryBot-Result+1 (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/514835. |
Message from Ian Lance Taylor: Patch Set 2: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/514835. |
Change-Id: If4c43c6759dd83e6df0e59b1890ca7fab0af34e5
This PR (HEAD: adc3041) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/514835. Important tips:
|
Message from Mateusz Poliwczak: Patch Set 3: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/514835. |
Message from Ian Lance Taylor: Patch Set 3: Auto-Submit+1 Code-Review+2 Run-TryBot+1 (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/514835. |
Message from Gopher Robot: Patch Set 3: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/514835. |
Message from Gopher Robot: Patch Set 3: TryBot-Result+1 (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/514835. |
For a while now we support EDNS, but the current implementation only sends the OPT resource and doesn't do anything with the response OPT resource. For reference the miekg/dns updates the RCode in the header when there is a OPT resource: https://github.com/miekg/dns/blob/48f38ebef989eedc6b57f1869ae849ccc8f5fe29/msg.go#L868-L872 Change-Id: I0a7146aed3e50654f340a3925f48612561cb85f4 GitHub-Last-Rev: adc3041 GitHub-Pull-Request: #61695 Reviewed-on: https://go-review.googlesource.com/c/go/+/514835 TryBot-Result: Gopher Robot <gobot@golang.org> Reviewed-by: David Chase <drchase@google.com> Run-TryBot: Ian Lance Taylor <iant@google.com> Reviewed-by: Ian Lance Taylor <iant@google.com> Auto-Submit: Ian Lance Taylor <iant@google.com>
This PR is being closed because golang.org/cl/514835 has been merged. |
|
For a while now we support EDNS, but the current
implementation only sends the OPT resource and doesn't
do anything with the response OPT resource.
For reference the miekg/dns updates the RCode in the
header when there is a OPT resource:
https://github.com/miekg/dns/blob/48f38ebef989eedc6b57f1869ae849ccc8f5fe29/msg.go#L868-L872