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

Can't use a string hostname as the server address #39

Closed
connection-reset opened this issue Jan 24, 2022 · 4 comments
Closed

Can't use a string hostname as the server address #39

connection-reset opened this issue Jan 24, 2022 · 4 comments
Labels
enhancement New feature or request

Comments

@connection-reset
Copy link

Stylophone 2.2.0 does not connect to my mpd server when I enter its hostname or IPv6 address, it fails with:

Couldn't find an MPD server at this address!

When I enter the IPv4 address is connects immediately.

DNS works and resolves both IPv4 and IPv6 addresses. MPD listens on both addresses.

@Difegue Difegue added the enhancement New feature or request label Jan 24, 2022
@Difegue
Copy link
Owner

Difegue commented Jan 27, 2022

Indeed, the underlying TcpClients used by MpcNET were initialized with the old IPv4 config, so it makes sense IPv6 wouldn't work 😅

I've modified the library code (Difegue/MpcNET@6bcc61b) to use dual mode sockets, so this should be solved in a future release.

@chowbok
Copy link

chowbok commented Nov 30, 2022

Specifying hostname still is not working. If this isn't an easy fix, then maybe just put a note in the dialog that it has to be an IP?

@Difegue
Copy link
Owner

Difegue commented Nov 30, 2022

Hmm I'm not sure if I've tried hostnames, the fix above was mostly for IPv6. Maybe hostnames aren't being translated to an IPEndPoint correctly!
Might as well reopen this with a name change..

@Difegue Difegue reopened this Nov 30, 2022
@Difegue Difegue changed the title Hostname must be IPv4 address Can't use a string hostname as the server address Nov 30, 2022
@Difegue
Copy link
Owner

Difegue commented Sep 15, 2023

Support for this should be available in the next update!

@Difegue Difegue closed this as completed Sep 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants