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.

Sign up for GitHub

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

Jump to bottom

net: dns host should be case insensitive in tests #34781

Closed
mengzhuo opened this issue Oct 9, 2019 · 4 comments
Closed

net: dns host should be case insensitive in tests #34781

mengzhuo opened this issue Oct 9, 2019 · 4 comments
Labels
FrozenDueToAge help wanted NeedsFix The path to resolution is known, but the work has not been done. Testing An issue that has been verified to require only test changes, not just a test failure.
Milestone
Unplanned

Comments

@mengzhuo
Copy link
Contributor

mengzhuo commented Oct 9, 2019

According to rfc4343:

Each node in the DNS tree has a name consisting
   of zero or more labels [STD13, RFC1591, RFC2606] that are treated in
   a case insensitive fashion.

What version of Go are you using (go version)?

$ go version
https://github.com/golang/go/commit/2197321db1dd997165c0091ba2bcb3b6be7633d0

Does this issue reproduce with the latest release?

Yes

What did you do?

https://build.golang.org/log/47cd658d0b040490e53ec53dc82e0032f0165b99

What did you expect to see?

ALL TESTES PASSED

What did you see instead?

--- FAIL: TestLookupGmailMX (0.02s)
    lookup_test.go:151: got &{gmail-smtp-in.l.google.COM. 5}; want a record containing google.com.
    lookup_test.go:151: got &{alt1.gmail-smtp-in.l.google.COM. 10}; want a record containing google.com.
    lookup_test.go:151: got &{alt2.gmail-smtp-in.l.google.COM. 20}; want a record containing google.com.
    lookup_test.go:151: got &{alt3.gmail-smtp-in.l.google.COM. 30}; want a record containing google.com.
    lookup_test.go:151: got &{alt4.gmail-smtp-in.l.google.COM. 40}; want a record containing google.com.
@odeke-em
Copy link
Member

odeke-em commented Oct 9, 2019

Thank you for this report @mengzhuo! So we have a recently closed issue #34446, fixed by CL https://golang.org/cl/196801 which was for case insensitivity of net.TestLookupGmailNS. /cc @bradfitz

@bradfitz bradfitz added help wanted NeedsFix The path to resolution is known, but the work has not been done. Testing An issue that has been verified to require only test changes, not just a test failure. labels Oct 9, 2019
@bradfitz bradfitz added this to the Unplanned milestone Oct 9, 2019
@gopherbot
Copy link

Change https://golang.org/cl/200277 mentions this issue: net: use case-insensitive host string comparison in TestLookup*

@Kroustille
Copy link

Can I take a look at this one ?

@ianlancetaylor
Copy link
Contributor

@Sehsyha Thanks, but the previous comment says that a patch has already been sent for this issue.

@golang golang locked and limited conversation to collaborators Oct 11, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
FrozenDueToAge help wanted NeedsFix The path to resolution is known, but the work has not been done. Testing An issue that has been verified to require only test changes, not just a test failure.
Projects
None yet
Milestone
Unplanned
Development

No branches or pull requests
6 participants
@bradfitz @mengzhuo @ianlancetaylor @odeke-em @Kroustille @gopherbot