Skip to content

Looking up ip for localhost should return 127.0.0.1 #24593

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

hoodmane
Copy link
Collaborator

Causing a failure in the Python test suite.

hoodmane added a commit to hoodmane/cpython that referenced this pull request Jun 17, 2025
It's failing because `gethostbyname_r()` is returning an incorrect
ip address for `localhost`. Will be resolved by upstream PR:
emscripten-core/emscripten#24593
@sbc100
Copy link
Collaborator

sbc100 commented Jun 17, 2025

I think I tried to make this same patch once and ran into issue.. I can't remember exactly where now..

freakboy3742 pushed a commit to python/cpython that referenced this pull request Jun 18, 2025
Emscripten currently `gethostbyname_r()` returns an incorrect
IP address for `localhost`. Will be resolved by upstream PR:
emscripten-core/emscripten#24593
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jun 18, 2025
…honGH-135634)

Emscripten currently `gethostbyname_r()` returns an incorrect
IP address for `localhost`. Will be resolved by upstream PR:
emscripten-core/emscripten#24593
(cherry picked from commit 2a49c54)

Co-authored-by: Hood Chatham <roberthoodchatham@gmail.com>
freakboy3742 pushed a commit to python/cpython that referenced this pull request Jun 18, 2025
…-135634) (#135651)

Emscripten currently `gethostbyname_r()` returns an incorrect
IP address for `localhost`. Will be resolved by upstream PR:
emscripten-core/emscripten#24593
(cherry picked from commit 2a49c54)

Co-authored-by: Hood Chatham <roberthoodchatham@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants