Skip to content

Add region tld - #256

Open
SaltySnail wants to merge 7 commits into
masterfrom
add-region-tld
Open

Add region tld#256
SaltySnail wants to merge 7 commits into
masterfrom
add-region-tld

Conversation

@SaltySnail

Copy link
Copy Markdown
Collaborator

This PR adds the ability to specify a region in either the .cfg file, or as a startup argument (which take presedence).


By creating this pull request, I understand that code that is AI generated or otherwise automatically generated may be rejected without further discussion.
I declare that I fully understand all code I pushed into this PR, and wrote all this code myself and own the rights to this code.

@SaltySnail
SaltySnail requested a review from WiserTixx June 23, 2026 00:43

@WiserTixx WiserTixx left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does the cli argument overwrite the config file when both are specified? If so besides my comment it lgtm

Comment thread include/Utils.h Outdated
@SaltySnail

Copy link
Copy Markdown
Collaborator Author

Does the cli argument overwrite the config file when both are specified? If so besides my comment it lgtm

Yes CLI overwrites the config file (config file is loaded before CLI is handled)

Comment thread src/Options.cpp Outdated
Comment thread src/Network/Core.cpp Outdated
Comment thread src/Network/Core.cpp Outdated
Comment thread src/Network/RegionHandler.cpp Outdated
Comment thread src/Network/RegionHandler.cpp Outdated
Comment thread include/Utils.h Outdated
Comment thread src/Network/Http.cpp Outdated
Comment thread src/Network/Core.cpp Outdated
Comment thread src/Network/RegionHandler.cpp
@SaltySnail
SaltySnail requested a review from WiserTixx August 8, 2026 19:48
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.

4 participants