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

on invalid IPv4 addresses, assume it's a hostname and do a DNS lookup #20

Merged
merged 3 commits into from Aug 10, 2022

Conversation

Istador
Copy link
Contributor

@Istador Istador commented Jul 10, 2022

Enable the full keyboard to enter non-numbers and increase max length from 15 to 50 for longer hostnames.

I know FQDNs can be longer than 50 characters, but that's less common (and painful to type on the Switch).

Enable the full keyboard to enter non-numbers and increase max length from 15 to 50 for longer hostnames.

I know FQDNs can be longer than 50 characters, but that's less common (and painful to type on the Switch).
This might otherwise cause confusion with the server.json setting, which should stay at 0.0.0.0.

127.0.0.1 might at least work with emulators running on the same PC out of the box. Whereas 0.0.0.0 should be invalid (except on some systems that map
it to localhost).
@Istador
Copy link
Contributor Author

Istador commented Jul 23, 2022

I bought the smoo.it domain.

With this PR the following domains can be entered in-game to join the public servers:

  • piplup.smoo.it
  • sanae.smoo.it
  • parknich.smoo.it
  • fruityloops.smoo.it
  • f0c0s.smoo.it
  • beni.smoo.it

Server overview: https://smoo.it/#/servers

@CraftyBoss CraftyBoss merged commit d3b1935 into CraftyBoss:dev Aug 10, 2022
@CraftyBoss
Copy link
Owner

I really wanted to have a system like this originally, thanks for figuring out a good solution!

@Istador Istador deleted the dns branch August 11, 2022 00:21
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.

None yet

2 participants