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

Explain the zonefile example better #395

Merged
merged 2 commits into from
Oct 24, 2024
Merged

Conversation

melroy89
Copy link
Contributor

Most people who read the example zone file for the fist time are new (like me).

I believe adding these three notes would help a lot to get people started. And remove any confusion.

Copy link
Member

@wcawijngaards wcawijngaards left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The additional notes are correct. But the NSD documentation is not really a guide for how to use DNS. So I can put this in, but realistically this is not an introductory text to explain how that DNS works.

@wcawijngaards wcawijngaards merged commit f958b2c into NLnetLabs:master Oct 24, 2024
4 checks passed
wcawijngaards added a commit that referenced this pull request Oct 24, 2024
- Merge #395: Explain the zonefile example better.
@melroy89
Copy link
Contributor Author

The additional notes are correct. But the NSD documentation is not really a guide for how to use DNS. So I can put this in, but realistically this is not an introductory text to explain how that DNS works.

Understood, but especially the missing IN was very confusing for me. In fact, I thought the example was wrong at first. Until I read about the implicit record type behavior. Thanks for merging any ways!

@melroy89 melroy89 deleted the patch-2 branch October 24, 2024 12:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants