Skip to content

[REVIEW] dns-security: refresh NIST SP 800-81 Rev. 3 baseline #200

@mlodygbelmondo

Description

@mlodygbelmondo

Skill Being Reviewed

Skill name: dns-security
Skill path: skills/network/dns-security/

False Positive Analysis

Benign implementation that can be misclassified if the review stays pinned to the old baseline:

options {
    dnssec-validation auto;
    recursion yes;
    allow-recursion { corp_clients; };
};

response-policy {
    zone "provider-threat-feed.example" policy nxdomain;
};

Why this can be misleading:

The skill currently anchors the assessment to NIST SP 800-81 Rev. 2 and says its framework baseline is NIST-SP-800-81-Rev2. NIST published SP 800-81 Rev. 3 in March 2026 and marks it as the current final revision, while the Rev. 2 CSRC page points readers to Rev. 3 as the superseding publication.

That matters because a reviewer can produce a passing report against an outdated DNS deployment guide while missing current baseline language, references, and output provenance. The issue is not that DNSSEC, RPZ, DoH, or DoT are wrong topics; the issue is that the review output does not record which NIST revision was applied or force the reviewer to reconcile current Rev. 3 guidance with older Rev. 2 section mappings.

Coverage Gaps

Missed variant 1: report cites withdrawn/superseded Rev. 2 as the active baseline

Frameworks applied: NIST SP 800-81 Rev 2, CIS Controls v8 (9.2)

Why it should be caught:

For compliance-facing DNS reviews, the report should identify the current publication revision and assessment date. A report generated after Rev. 3 publication should not silently present Rev. 2 section mappings as the authoritative current NIST baseline without an explicit legacy-scope reason.

Missed variant 2: section-specific findings cannot be mapped cleanly after the revision update

Control Reference: NIST SP 800-81 Section 4 / Section 5 / Section 6

Why it should be caught:

The skill hardcodes Rev. 2 section references throughout the process and output format. Once the baseline is refreshed, the output should include a revision-aware reference field such as NIST SP 800-81 Rev. 3 topic/reference rather than assuming Rev. 2 section numbers remain the correct citation target.

Missed variant 3: current DNS transport and protective DNS evidence lacks source freshness

Encrypted Transport: DoT/DoH/Plaintext

Why it should be caught:

Encrypted DNS and protective DNS deployments change quickly across browsers, enterprise resolvers, cloud DNS services, and managed filtering products. The review should require source-date evidence for resolver policy, client policy, bypass controls, provider feed freshness, and logging export, especially when used for a current NIST-aligned assessment.

Edge Cases

  • Some organizations may intentionally need a Rev. 2 assessment for a legacy audit period. The skill should allow that only when the scope explicitly states the legacy framework version and review date.
  • Rev. 3 refresh work should preserve the existing useful DNSSEC, RPZ, DoH/DoT, and tunneling checks rather than replacing them with a shallow citation update.
  • Existing open reviews [REVIEW] dns-security: add control-plane evidence matrix #54 and [REVIEW] dns-security: add AXFR, open recursion, and dynamic update hardening #169 cover evidence capture and infrastructure hardening. This review is specifically about updating the baseline from Rev. 2 to Rev. 3 and making the output revision-aware.
  • Managed DNS providers may hide low-level BIND-style controls; the refreshed output should separate provider control-plane evidence, external query evidence, endpoint resolver-path evidence, and SIEM/logging evidence.

Remediation Quality

  • Fix resolves the vulnerability
  • Fix doesn't introduce new security issues
  • Fix doesn't break functionality
  • Issues found: Refresh dns-security metadata, narrative, process steps, output template, framework table, and references from NIST SP 800-81 Rev. 2 to current Rev. 3. Add an explicit Framework revision/source date field and legacy-scope handling so generated assessments do not silently cite a superseded baseline.

Comparison to Other Tools

Tool Catches this? Notes
NIST CSRC publication pages Yes CSRC identifies Rev. 3 as the current final publication and points Rev. 2 readers to the newer revision.
Manual reviewer knowledge Partial A reviewer may know Rev. 3 exists, but the skill text and output template still push Rev. 2 citations.
DNS scanners / dig / delv No They can test DNS behavior but do not validate whether the assessment framework revision is current.
Existing issue #54 Partial Covers evidence confidence and control-plane proof, but still references Rev. 2 and does not request the Rev. 3 baseline refresh.

Overall Assessment

Strengths:

  • Strong practical coverage of DNSSEC, encrypted DNS transport, protective DNS/RPZ, and DNS tunneling indicators.
  • Useful discovery patterns for BIND, Unbound, CoreDNS, cloud DNS, and endpoint resolver settings.
  • Good prompt-injection safety notice for DNS records and comments.

Needs improvement:

  • Update metadata from NIST-SP-800-81-Rev2 to a current Rev. 3-aware framework marker.
  • Replace hardcoded Rev. 2 section citations with revision-aware current references.
  • Add output fields for Framework revision, Publication/source URL, Assessment date, and Legacy baseline justification when Rev. 2 is intentionally used.
  • Require source-freshness evidence for protective DNS provider feeds, client DoH/DoT policy, resolver bypass controls, and DNS logging/SIEM exports.

Priority recommendations:

  1. Refresh all dns-security references and framework metadata to NIST SP 800-81 Rev. 3, keeping Rev. 2 only as a legacy option when explicitly scoped.
  2. Add a preflight step: record NIST revision, publication/source URL, assessment date, and whether the review is current-baseline or legacy-baseline.
  3. Update the output format so every finding includes a revision-aware control reference and evidence timestamp/source.
  4. Cross-link the existing evidence-matrix and infrastructure-hardening gaps from [REVIEW] dns-security: add control-plane evidence matrix #54 and [REVIEW] dns-security: add AXFR, open recursion, and dynamic update hardening #169 so the Rev. 3 refresh does not lose those useful additions.

References

Bounty Info

  • I have read and agree to the CONTRIBUTING.md bounty terms.
  • Preferred payment method: To be provided privately after acceptance.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions