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

DnsServerEDns: New resource proposal #194

Closed
johlju opened this issue Mar 1, 2021 · 0 comments · Fixed by #224
Closed

DnsServerEDns: New resource proposal #194

johlju opened this issue Mar 1, 2021 · 0 comments · Fixed by #224
Labels
resource proposal The issue is proposing a new resource in the resource module.

Comments

@johlju
Copy link
Member

johlju commented Mar 1, 2021

Description

Suggest adding a new resource that changes DNS server EDNS settings. Using the commands Get-DnsServerEDns and Set-DnsServerEDns.

Proposed properties

  • DnsServer <Key, String> - The host name of the DNS server to change, or use 'localhost' for the current node.
  • EnableProbes <Write, Boolean> - Specifies whether to enable the server to probe other servers to determine whether they support EDNS.
  • CacheTimeout <Write, String> - Specifies the number of seconds that the DNS server caches EDNS information. The default value is 604,800 seconds (one week).
  • EnableReception <Write, Boolean> - Specifies whether the DNS server accepts queries that contain an EDNS record.

Special considerations or limitations

The parameter CacheTimeout is of the type [String] that should be converted to [TimeSpan] similar to the TTL parameter here:
https://github.com/dsccommunity/xDnsServer/blob/dbcdb067ce905e7c3fc5f74d1ad787c3166f33ba/source/DSCResources/MSFT_xDnsRecordSrv/MSFT_xDnsRecordSrv.psm1#L200-L203

The parameters EnableProbes and CacheTimeout replaces the properties EnableEDnsProbes and EDnsCacheTimeout in the resource xDnsServerSetting, To avoid ping-pong behavior those properties should be removed from the resource xDnsServerSetting once this issue is resolved.

@johlju johlju changed the title DnsServerScavenging: New resource proposal DnsServerEdns: New resource proposal Mar 1, 2021
@johlju johlju added help wanted The issue is up for grabs for anyone in the community. resource proposal The issue is proposing a new resource in the resource module. labels Mar 1, 2021
@johlju johlju changed the title DnsServerEdns: New resource proposal DnsServerEDns: New resource proposal Mar 1, 2021
@johlju johlju added in progress The issue is being actively worked on by someone. and removed help wanted The issue is up for grabs for anyone in the community. labels Mar 24, 2021
johlju added a commit that referenced this issue Mar 26, 2021
- xDnsServer
  - Added new resource
    - _DnsServerEDns_ - resource to enforce extension mechanisms for DNS
      (EDNS) settings (issue #194).
  - Added new private function `Assert-TimeSpan` to help assert that a value
    provided in a resource can be converted to a `[System.TimeSpan]` and
    optionally evaluates so it is not below a minium value or over a maximum
    value.
@johlju johlju removed the in progress The issue is being actively worked on by someone. label Mar 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
resource proposal The issue is proposing a new resource in the resource module.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant