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

Connectiontest doesn't work with IPv6 domain or address #4733

Open
bdgza opened this issue Jun 12, 2024 · 3 comments
Open

Connectiontest doesn't work with IPv6 domain or address #4733

bdgza opened this issue Jun 12, 2024 · 3 comments
Labels
Confirmed Bug The bug reported is confirmed and able to be replicated.

Comments

@bdgza
Copy link

bdgza commented Jun 12, 2024

Describe the bug

When using the connectiontest command with an IPv6 (only) domain name the error is:
Got the error message: sample.ipv6-only.domain: Name has no usable address

When using the connectiontest command with an IPv6 address directly, with or without [ ], it parses the : as the port separator and fails:
Not a valid port! Specify a valid numeric port.

To Reproduce

  1. Set up a domain name with an IPv6 record only

  2. Run the connectiontest command on that domain e.g., geyser connectiontest sample.ipv6-only.domain:19132

  3. Run the connectiontest command on an IPv6 address e.g., geyser connectiontest [2001:0000:130F:0000:0000:09C0:876A:130B]:19132

Expected behaviour

The connectiontest should be performed via IPv6 and pass or fail depending on if the connection to the server was tested successfully. It should not fail because it can't handle an IPv6 address.

Screenshots / Videos

No response

Server Version and Plugins

This server is running Paper version git-Paper-497 (MC: 1.20.4) (Implementing API version 1.20.4-R0.1-SNAPSHOT) (Git: d8d54d9 on ver/1.20.4)

AFKPlus 3.4.1
AFKPlusPrefix 1.3.5
AFKPlusRandomMessages 1.0.2
AntiCreeper 5.1
Backuper 2.0.2
BKCommonLib 1.20.4-v4
ChestCleaner 2.4.4
DatapackLoader 1.3.0
DiscordSRV 1.27.0
dynmap 3.7-beta-4-935
Dynmap-LoadedChunks 0.4
DynmapBannerMarkers 1.2.0
FileCleaner s2.0.0
floodgate 2.2.3-SNAPSHOT (b107-c4a4487)
Geyser-Spigot 2.3.2-SNAPSHOT
HoloMobHealth 2.3.7.0
ImageFrame 1.7.8.4
InfoHUD 1.5.3
ItemframeSnipper 1.0.0
LuckPerms 5.4.117
PlaceholderAPI 2.11.6
PluginManager 2.8.1
ProtocolLib 5.2.0-SNAPSHOT-679
spark 1.10.59
Train_Carts 1.20.4-v2
TrainCartsDestinationSelector 0.3
ViaVersion 4.10.2

Geyser Dump

https://dump.geysermc.org/CnXs2hZWjfHmLY2WLzCQugJk2Mp3V80H

Geyser Version

2.3.2-SNAPSHOT

Minecraft: Bedrock Edition Device/Version

No response

Additional Context

I am running a server behind a DSLite internet connection so only IPv6 port sharing is available to me for hosting.

@onebeastchris
Copy link
Member

Regarding testing IPv6 addresses with a port, that'll be resolved by #3808 as it splits ip:port combo into separate fields. As for domains, that may be an issue with the server ping api we're using, I'll ask around.

@onebeastchris onebeastchris added the Confirmed Bug The bug reported is confirmed and able to be replicated. label Jun 12, 2024
@onebeastchris
Copy link
Member

#4862 includes the cloud changes which changed the connectiontest command to split ip and port - this should allow testing ipv6 addresses now. Please let us know if this doesnt work despite it!

@bdgza
Copy link
Author

bdgza commented Aug 1, 2024

I ran the connectiontest with my IPv6 domain and it ran without error, but the result was negative/offline. However, since the domain is IPv6 only I am not sure if it took the actual IPv6 address, or ran with some faulty IPv4 instead – there's no feedback on any domain resolution so I can't be sure.

Thus, I ran the connectiontest again with the raw full IPv6 address and it doesn't work:

geyser connectiontest xxxx:xxxx:xxxx:xxxx:xxxx:xxxx:xxxx:xxxx 19132

[Thu 06:27:05 INFO  Server] Expected whitespace to end one argument, but found trailing data
...ntest xxxx:xxxx:xxxx:xxxx:xxxx:xxxx:xxxx:xxxx 19132<--[HERE]

Edit: I also tested my server on mcsrvstat.us and both the IPv6 domain as well as the raw IPv6 address return success/online

@onebeastchris onebeastchris reopened this Aug 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Confirmed Bug The bug reported is confirmed and able to be replicated.
Projects
None yet
Development

No branches or pull requests

2 participants