dsccommunity / DnsServerDsc

This module contains DSC resources for the management and configuration of Windows Server DNS Server.
MIT License
65 stars 54 forks source link

DnsServerRecursion: New resource proposal #198

Closed johlju closed 3 years ago

johlju commented 3 years ago

Description

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

Proposed properties

Special considerations or limitations

The default zone '.' that is set with Set-DnsServerRecursionScope is the same scope that is set with this resource. So in a future resource DnsServerRecursionScope it should prevent from changing the property EnableRecursion for the default scope ('.') and instead refer to use this resource.

The parameters Enable, RetryInterval, Timeout, and SecureResponse replaces the properties NoRecursion, RecursionRetry, RecursionTimeout, and SecureResponse respectively in the resource xDnsServerSetting, To avoid ping-pong behavior those properties should be removed from the resource xDnsServerSetting once this issue is resolved.

Sudman1 commented 3 years ago

Using the commands Get-DnsServerCache and Set-DnsServerCache.

I believe you meant Get-DnsServerRecursionScope and Set-DnsServerRecursionScope instead of the cache cmdlets.

johlju commented 3 years ago

@Sudman1 good catch. Copy paste mistake, it was meant to be Get-DnsServerRecursion and Set-DnsServerRecursion. Updated now. The ones you mentioned is tracked in PR #199.

johlju commented 3 years ago

The propertySecureResponse changes the same value as EnablePollutionProtection in DnsServerCache does. I think we should leave this property out of this resource and document that the resource DnsServerCache should be used to enforce pollution protection.