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

hostip: don't use the resolver for FQDN localhost #15676

Closed
wants to merge 1 commit into from

Conversation

jay
Copy link
Member

@jay jay commented Dec 3, 2024

  • Treat [<any>.]localhost. as fixed value 127.0.0.1 and ::1 instead of querying the resolver.

Prior to this change, b5c0fe2 (precedes 7.85.0) did the same for non-FQDN <any>.localhost.

Prior to this change, 1a0ebf6 (precedes 7.78.0) did the same for non-FQDN localhost.

Ref: #15628

Closes #xxxxx

- Treat `[<any>.]localhost.` as fixed value 127.0.0.1 and ::1 instead
  of querying the resolver.

Prior to this change, b5c0fe2 (precedes 7.85.0) did the same for
non-FQDN `<any>.localhost`.

Prior to this change, 1a0ebf6 (precedes 7.78.0) did the same for
non-FQDN `localhost`.

Ref: curl#15628

Closes #xxxxx
@jay jay added the name lookup DNS and related tech label Dec 3, 2024
@testclutch
Copy link

Analysis of PR #15676 at b52f8c31:

Test 283 failed, which has NOT been flaky recently, so there could be a real issue in this PR.

Test 1242 failed, which has NOT been flaky recently, so there could be a real issue in this PR.

Generated by Testclutch

@jay jay closed this Dec 5, 2024
@jay jay deleted the more_localhost_hardcode branch December 5, 2024 22:15
jay added a commit that referenced this pull request Dec 5, 2024
- Treat `[<any>.]localhost.` (ie localhost with FQDN period terminator)
  as fixed value 127.0.0.1 and ::1 instead of querying the resolver.

Prior to this change, b5c0fe2 (precedes 7.85.0) did the same for
non-FQDN `<any>.localhost`.

Prior to this change, 1a0ebf6 (precedes 7.78.0) did the same for
non-FQDN `localhost`.

Ref: #15628 (comment)

Closes #15676
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
name lookup DNS and related tech
Development

Successfully merging this pull request may close these issues.

3 participants