Add "RegisterIPv6" setting#495
Conversation
|
Those lines are not useless, not everything supports ipv6 (in beammp and for users). Without this line all servers on the server list that support ipv6 will change their listing ip to ipv6. I do not mind this being an option to server operators but I'd prefer it to be toggled via an environment variable. E.g. |
|
libcurl is smart enough to use both ipv4 and ipv6. i've just now tested beammp server in an ipv4 only container, and it has no problem connecting to the backend with this patch. so yes, im pretty sure restricting to ipv4 only is useless. |
|
That's not my point. I never said curl was incapable of handling that. |
|
I'm failing to replicate the scenario you are describing. I created a dual-stack container while also dropping all out-going IPv6 packets (the container/server has an ipv6 address, but one that doesn't work). The server still works. So unless I'm yet again misinterpreting you, it seems that the server has no problem falling back to ipv4 when ipv6 is assigned but not working. |
|
I'm not talking about the server, it works just fine. When a server registers on the backend with ipv6, and a client without ipv6 support, either because the BeamMP mod/Launcher doesn't support it, or because the user's router/ISP doesn't support it. It will fail, without prior notice. For this to be resolved sizeable changes would need to be made to the backend. |
|
I think I understand now. The best solution would be to attempt to register both address families with the backend, so that a dual-stack server will work with as many users as possible without further configuration. I would take a crack at this myself, but I don't think the backend is open-source. |
|
But for now, I've added a new setting to solve my problem. Default behaviour is identical to pre-patch. |
these 2 lines of code are seemingly useless; removing them makes my ipv6 only beammp container work
By creating this pull request, I understand that code that is AI generated or otherwise automatically generated may be rejected without further discussion.
I declare that I fully understand all code I pushed into this PR, and wrote all this code myself and own the rights to this code.