-
Notifications
You must be signed in to change notification settings - Fork 92
Update DNS SRV documentation #1117
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: next-release
Are you sure you want to change the base?
Conversation
And add description of new places where SRV resolution is used in 3.12.0
9df5a4b to
781c21c
Compare
781c21c to
cdcdf99
Compare
|
OK, I've reorganised the DNS SRV page to eliminate redundancy and provide a more logical order of presentation. Hopefully it might conceivably pass muster! |
| ~~~ | ||
|
|
||
| Alternatively, `server1.example.com` can be a router, NAT-gateway, or load-balancer that forwards each of the ports to a backend Jamulus Server (or Servers). | ||
| In the case of Jamulus, the `service` must be `jamulus` and the `proto` must be `udp`, so the record will look like this: |
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.
so the record will look like this
I'm not sure using jamulus.example.com as the host is great. Maybe service myserver.example.com and host host.example.com, allowing connection to myserver.example.com without the port being needed and that the Jamulus Server is running on host.example.com - to make it really clear.
_jamulus._udp.myserver.example.com. 60 IN SRV 0 5 12345 host.example.com.
Short description of changes
Move documentation for DNS SRV from Unregistered Servers to its own page.
Expand the DNS SRV page.
Add references to the new page from the Running a Server and Unregistered Servers pages.
Context: Fixes an issue? Related issues
Provides supporting documentation for jamulussoftware/jamulus#3556
Status of this Pull Request
Tested in Jekyll and ready for review.
What is missing until this pull request can be merged?
Only review.
Does this need translation?
YES
Checklist