-
Notifications
You must be signed in to change notification settings - Fork 18.5k
net: report detailed DNS errors with Extended DNS Errors #63201
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
base: master
Are you sure you want to change the base?
Conversation
784ecc5 to
9007d6c
Compare
|
This PR (HEAD: 9007d6c) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/530876. Important tips:
|
|
Message from Mateusz Poliwczak: Patch Set 1: Run-TryBot+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/530876. |
Change-Id: If48df42831c960b1edd680aa0d0f63dc9becd900
9007d6c to
f54fb07
Compare
|
Message from Gopher Robot: Patch Set 1: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/530876. |
|
Message from Gopher Robot: Patch Set 1: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/530876. |
|
This PR (HEAD: f54fb07) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/530876. Important tips:
|
|
Message from Gopher Robot: Patch Set 1: TryBot-Result-1 (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/530876. |
|
Message from Mateusz Poliwczak: Patch Set 2: Run-TryBot+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/530876. |
Change-Id: I649ebb968f70f45b3f2843f464e84484b4b4f8a3
|
Message from Gopher Robot: Patch Set 2: (2 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/530876. |
|
This PR (HEAD: ba881da) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/530876. Important tips:
|
|
Message from Gopher Robot: Patch Set 2: TryBot-Result+1 (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/530876. |
|
Message from Mateusz Poliwczak: Patch Set 3: Run-TryBot+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/530876. |
|
Message from Gopher Robot: Patch Set 3: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/530876. |
|
Message from Gopher Robot: Patch Set 3: TryBot-Result+1 (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/530876. |
Before, when the query failed we returned a simple "server misbehaving"
error, with this change the go resolver will report detailed errors
with the use of the EDE extension found in the OPT resource.
Given that resolvers are now starting to support this EDNS(0) option,
I think it is worth to support it in the pure go resolver.
Fixes #63192