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

fix: bind to [::1] per default instead of 127.0.0.1 in PocketIC #3280

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

mraszyk
Copy link
Contributor

@mraszyk mraszyk commented Dec 20, 2024

This PR makes the PocketIC server and HTTP gateway bind to [::1] per default instead of 127.0.0.1 to work around an issue with macOS sandbox spuriously preventing binding to 127.0.0.1. Because fqdn! panics with [::1], FQDN::from_str is now used instead.

@mraszyk mraszyk requested a review from a team as a code owner December 20, 2024 12:42
@github-actions github-actions bot added the fix label Dec 20, 2024
@mraszyk mraszyk added the CI_MACOS_INTEL Run Bazel Test macOS Intel CI job which runs only on push to master label Dec 20, 2024
@mraszyk mraszyk marked this pull request as draft December 20, 2024 13:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI_MACOS_INTEL Run Bazel Test macOS Intel CI job which runs only on push to master fix @pocket-ic
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant