Skip to content

[no sq] Enable ipv6_server by default #15739

Merged
sfan5 merged 2 commits into
masterfrom
ipv6
Feb 15, 2025
Merged

[no sq] Enable ipv6_server by default #15739
sfan5 merged 2 commits into
masterfrom
ipv6

Conversation

@sfan5

@sfan5 sfan5 commented Jan 31, 2025

Copy link
Copy Markdown
Member

This makes it so that if you have IPv6 and enable server announcements you automatically arrive at a configuration that actually works (domain or not).
Previously if you had IPv6 configured on your server and maybe even a domain, the server list would reject you because the server can't actually be reached over IPv6 by default.

Implications:

  • the IP returned in get_player_information will look like ::ffff:1.2.3.4 by default. I think most mods handle this.

Note: this PR was written on a train.

To do

This PR is a Ready for Review.

@sfan5 sfan5 added @ Server / Client / Env. Feature ✨ PRs that add or enhance a feature labels Jan 31, 2025
@sfan5 sfan5 added this to the 5.12.0 milestone Feb 3, 2025
@sfan5 sfan5 added the Rebase needed The PR needs to be rebased by its author label Feb 14, 2025
@sfan5 sfan5 removed the Rebase needed The PR needs to be rebased by its author label Feb 15, 2025
@sfan5 sfan5 merged commit d027fc9 into master Feb 15, 2025
@sfan5 sfan5 deleted the ipv6 branch February 15, 2025 11:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants