Fix ednscachetimeout range/default in dnscmd docs#8114
Open
sa1181405 wants to merge 1 commit intoMicrosoftDocs:mainfrom
Open
Fix ednscachetimeout range/default in dnscmd docs#8114sa1181405 wants to merge 1 commit intoMicrosoftDocs:mainfrom
sa1181405 wants to merge 1 commit intoMicrosoftDocs:mainfrom
Conversation
Contributor
|
@sa1181405 : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change. |
Contributor
|
Learn Build status updates of commit b2a38f6: ✅ Validation status: passed
For more details, please refer to the build report. |
Contributor
There was a problem hiding this comment.
Pull request overview
This PR corrects the documented valid range and default value for the ednscachetimeout parameter in the dnscmd documentation to match the actual implementation in the DNS Server.
Changes:
- Updated the valid range from 3600-15724800 seconds to 10-86400 seconds
- Corrected the default value from 604800 seconds (one week) to 900 seconds (15 minutes)
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Contributor
|
#assign: @robinharwood, @dknappettmsft Can you review the proposed changes? #label:"aq-pr-triaged" @MicrosoftDocs/public-repo-pr-review-team |
Contributor
|
Users robinharwood are already assigned. |
prmerger-automator
bot
added
the
aq-pr-triaged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The dnscmd documentation currently lists an incorrect valid range (and default) for
ednscachetimeout:Currently documented: 3600-15724800 seconds (default 604800 / one week)
In practice, dnscmd enforces 10-86400 seconds (default 900 / 15 minutes)
Values > 86400 fail with
DNS_ERROR_DWORD_VALUE_TOO_LARGEValues < 10 fail with
DNS_ERROR_DWORD_VALUE_TOO_SMALLThis PR updates the dnscmd docs to match the authoritative DNS Server Integer Properties documentation:
dnscmd docs: https://learn.microsoft.com/en-us/windows-server/administration/windows-commands/dnscmd
DNS Server Integer Properties: https://learn.microsoft.com/en-us/openspecs/windows_protocols/ms-dnsp/8903e50a-9183-4a7d-9640-53f6f5a91481