README: fix NS address in test command #115
Conversation
| @@ -205,7 +205,7 @@ Note: The `txt` field must be exactly 43 characters long, otherwise acme-dns wil | |||
|
|
|||
| 4) Perform a DNS lookup to the test subdomain to confirm that everything is working properly: | |||
| ``` | |||
| $ dig @ns.auth.example.org d420c923-bbd7-4056-ab64-c3ca54c9b3cf.auth.example.org | |||
| $ dig @auth.example.org d420c923-bbd7-4056-ab64-c3ca54c9b3cf.auth.example.org | |||
There was a problem hiding this comment.
actually the @auth.example.org should not be required at all. If the DNS setup is correct, this server should be queried eventually anyway.
There was a problem hiding this comment.
That's correct. And for testing purposes, even more usable actually. The current format is probably more usable for troubleshooting scenarios.
In general, I feel that the README.md should get restructured, and internally linked. For example, installation should link to the DNS configuration section, and testing it out to the (hopefully upcoming) troubleshooting section. Ideally these should follow each other in the document itself. I'll create an issue for that.
|
Thanks for fixing this, I'll merge it now, but the whole documentation should go through some rewriting soonish. I added some documentation development thoughts to the code comment above, and will create a new issue for those. |
The address of the nameserver changed when the README was made consistent with the current config file. This PR simply corrects the NS address in a test command.