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

Do not fail silently if IP in connection window is invalid #1945

Open
ann0see opened this issue Aug 8, 2021 · 8 comments
Open

Do not fail silently if IP in connection window is invalid #1945

ann0see opened this issue Aug 8, 2021 · 8 comments
Labels
bug Something isn't working

Comments

@ann0see
Copy link
Member

ann0see commented Aug 8, 2021

Follow up #1938 by @softins

If an invalid IP is entered, the connection window closes after clicking on connect and Jamulus does not give an error message.

Proposed behaviour:

  1. Check IP/Domain on entry and give a red warning "The IP you entered is invalid. Make sure you enter it in a correct format"
  2. Same as above, but after clicking on connect as error message
@gilgongo
Copy link
Member

gilgongo commented Nov 7, 2021

It's really a bug, so I'll flag for next release in hope.

@gilgongo gilgongo added this to the Release 3.9.0 milestone Nov 7, 2021
@pljones
Copy link
Collaborator

pljones commented Nov 7, 2021

#1938 is rather long. Was there any particular comment?

By "Invalid", does this mean "Syntactically incorrect", rather than "Cannot connect to a Jamulus server at that address"?

If it's the former, it should be before closing the Connect dialog. If it's the latter, the existing warning banner should suffice.

@gilgongo
Copy link
Member

gilgongo commented Nov 7, 2021

@pljones I think invalid as in syntactically incorrect (so 192.168.o.10, and hopefully also a DNS address that doesn't resolve) as the message is "Make sure you enter it in a correct format". Should probably be "The address you entered is invalid..." I think.

@pljones
Copy link
Collaborator

pljones commented Nov 7, 2021

Yes. (Ugh, took a while to read your message correctly...)

It's got to cope with non-IP addresses, of course. "I0.10.123.I0" is a perfectly valid address syntactically (it probably doesn't resolve, so you won't connect to anything).

The easiest check is to do just that - attempt to resolve the input text to an IP address (with optional port).

If it fails, you error: "The address entered is not valid. Make sure you typed it correctly."

If it works, you exit the Connect dialog and proceed to attempting to communicate with the provided network address.

@ann0see ann0see removed this from the Release 3.9.0 milestone Jun 14, 2022
@ann0see
Copy link
Member Author

ann0see commented Jun 14, 2022

De-Tagged. I think it’s a thing for the backlog

@ann0see ann0see added this to Tracking Jul 1, 2023
@github-project-automation github-project-automation bot moved this to Triage in Tracking Jul 1, 2023
@pljones pljones added bug Something isn't working and removed feature request Feature request labels Aug 12, 2023
@pljones
Copy link
Collaborator

pljones commented Aug 12, 2023

I'm going to put this on the 3.11.0 backlog as it shouldn't be too hard.

@pljones pljones moved this from Triage to Backlog in Tracking Aug 12, 2023
@pljones pljones added this to the Release 3.11.0 milestone Aug 12, 2023
@ann0see
Copy link
Member Author

ann0see commented Aug 12, 2023

Probably we can check with a regular expression. I still believe the problem lies deeper: what if we enter an which doesn't have a Jamulus server running?
But maybe that's already handled correctly with a timeout.

@pljones pljones moved this from Backlog to Triage in Tracking Sep 27, 2023
@pljones pljones modified the milestones: Release 3.11.0, Release 3.12.0 May 3, 2024
@ann0see
Copy link
Member Author

ann0see commented Sep 22, 2024

I believe #3372 may fix this as side effect - to some extent at least.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Triage
Development

No branches or pull requests

3 participants