-
Notifications
You must be signed in to change notification settings - Fork 237
Provides SRV support for -e|--directoryaddress option #3556
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
base: main
Are you sure you want to change the base?
Conversation
|
I have just created SRV records in our zone on Cloudflare for the various directories, as follows: Once we are happy they are working correctly, we can wind the TTLs up from 1 minute to something longer. |
|
@softins thanks, I have reconfigured one of my servers to use SRV for anygenre2.jamulus.io, monitoring. |
softins
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks fine to me. Tested on all standard directories, using both -e and the Custom setting in the server GUI.
|
While the SRV lookups work and servers can connect to a directory server, I'm seeing all my test servers lose their ability to maintain their registrations eventually, some indeterminate time within 24 hrs, and attempts to re-register stop being logged. Any clients that were connected to the server itself also get disconnected. Attempting to investigate further... |
Short description of changes
Provides SRV DNS support for
-e|--directoryaddressoption.CHANGELOG: SKIP
Context
Currently one needs to provide both an IP/host and port number to the
-e|--directoryaddressoption if the directory server is not using the default port22124. This patch will enable the ability to use preconfigured SRV DNS records by a server to connect to a directory without having to provide a port number.The patch expands on SRV support in client code already in
main.I have created SRV DNS records to test with that point to each of the seven public directory servers provided by the Jamulus team:
You can confirm the SRV records using the following:
Mac/Linux
Windows
In order to utilize this functionality for the Jamulus public space, the Jamulus team could create the seven SRV records and publish those in the same table that displays the server host/port pairs in https://jamulus.io/wiki/Running-a-Server#registered-mode
Does this change need documentation? What needs to be documented and how?
Unsure. According to
--helpoutput, the option-c|--connectdoesn't mention anything about SRV support. In that same vein I submit nothing should be added for-e|--directoryaddresseither.Status of this Pull Request
What is missing until this pull request can be merged?
Checklist