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

Enable ipv6 client side support by default #3300

Open
stevie2sheds opened this issue Jun 24, 2024 · 3 comments
Open

Enable ipv6 client side support by default #3300

stevie2sheds opened this issue Jun 24, 2024 · 3 comments
Assignees
Labels
feature request Feature request
Milestone

Comments

@stevie2sheds
Copy link

Current behaviour and why should it be changed?

IPv6 client support currently has to be enabled at the command line using the -6 option. This requires the user either to edit the shortcuts/links or start the client from the command line. This varies in complexity depending on the OS, but it is a major barrier to IPv6 adoption when trying to support large groups having to make the required changes to client startup. Having IPv6 support enabled by default would remove this barrier.

Describe possible approaches
Another approach could be to enable IPv6 support using a checkbox within the UI. This would allow the user to elect to enable IPv6, and perhaps there is a very marginal resource saving in not starting a feature that may not be used.

Describe the solution you'd like
Enable ipv6 client side support by default with no additional checkbox in the UI. That solution means no action needs to be taken at the client side at all for IPv6.

Has this feature been discussed and generally agreed?

The topic has been discussed here: https://github.com/orgs/jamulussoftware/discussions/3291

@stevie2sheds stevie2sheds added the feature request Feature request label Jun 24, 2024
@softins
Copy link
Member

softins commented Jun 24, 2024

If we did make it a checkbox in Settings, it should also be stored in the ini file, to remember the setting.

In that case, rather than the complication of changing the open socket on the fly when the checkbox is toggled, it may well be sufficient to request a restart, like it does when the language setting is changed.

Would also need to make sure the ini file is loaded before the socket is created.

@stevie2sheds
Copy link
Author

That sounds as if the checkbox option is of greater complexity than enabling by default and therefore the always on option is preferable.

@pljones pljones added this to the Release 4.0.0 milestone Jun 29, 2024
@pljones
Copy link
Collaborator

pljones commented Jun 29, 2024

I've tagged this as Release 4.0.0 as it'll probably get picked up alongside #3242.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request Feature request
Projects
Status: Triage
Development

No branches or pull requests

3 participants