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: LookupHost("nonexistentdomain.golang.org"): returns [173.194.74.141 2607:f8b0:4001:c14::8d] #25361
Comments
Change https://golang.org/cl/112955 mentions this issue: |
The
It looks like the Plan 9 resolver follows the CNAME, while it doesn't seem to be the case on the other builders. Would it be possible to change TestLookupHostCancel to query a name that doesn't return a CNAME entry, like |
After re-reading the change, I figured out it doesn't matter if the DNS request succeeds or not. What matter is that when the DNS request is canceled, LookupHost returns errCanceled. This doesn't work on Plan 9 because DNS requests can't be canceled on Plan 9 currently. On Plan 9, DNS requests are done by writing to a file. Maybe this issue could be fixed by implementing cancellations in LookupHost on Plan 9. |
Same on Windows. DNS requests are answered by calling blocking API.
That is what windows code does. CL 111718 fixed a bug in that code. I will leave CL 112955 in, if you decide to submit it. After you convince Brad to remove -2. Alex |
Change https://golang.org/cl/112981 mentions this issue: |
I'll investigate the CNAME issue and make sure we're using a DNS name with no results too. |
Please keep some bug open about CNAME for that domain if you close this one. |
I've opened #25370. |
Recent CL 111718 added TestLookupHostCancel that fails on plan9 builder
https://build.golang.org/log/63c5911a1ae08505472b57f58267a3f778b3eb5a
/cc @0intro
Alex
The text was updated successfully, but these errors were encountered: