Skip to content

Conversation

@rgacogne
Copy link
Member

@rgacogne rgacogne commented Nov 3, 2025

Short description

As reported by Eddict (thanks!).

Fixes #16386.

Checklist

I have:

  • read the CONTRIBUTING.md document
  • read and accepted the Developer Certificate of Origin document, including the AI Policy, and added a "Signed-off-by" to my commits
  • compiled this code
  • tested this code
  • included documentation (including possible behaviour changes)
  • documented the code
  • added or modified regression test(s)
  • added or modified unit test(s)

As reported by Eddict (thanks!).

Signed-off-by: Remi Gacogne <[email protected]>
@coveralls
Copy link

coveralls commented Nov 3, 2025

Pull Request Test Coverage Report for Build 19031394433

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 32 unchanged lines in 11 files lost coverage.
  • Overall coverage increased (+0.08%) to 72.82%

Files with Coverage Reduction New Missed Lines %
pdns/dnsdistdist/dnsdist.cc 2 70.94%
pdns/recursordist/aggressive_nsec.cc 2 66.54%
pdns/recursordist/sortlist.cc 2 75.29%
modules/gpgsqlbackend/spgsql.cc 3 68.66%
pdns/axfr-retriever.cc 3 67.74%
pdns/opensslsigners.cc 3 61.48%
pdns/rcpgenerator.cc 3 90.87%
pdns/recursordist/recpacketcache.hh 3 91.04%
pdns/remote_logger.cc 3 58.04%
pdns/dnsdistdist/dnsdist-tcp.cc 4 77.55%
Totals Coverage Status
Change from base Build 19031300336: 0.08%
Covered Lines: 127131
Relevant Lines: 163882

💛 - Coveralls

@rgacogne rgacogne merged commit f583d31 into PowerDNS:master Nov 3, 2025
160 of 164 checks passed
@rgacogne rgacogne deleted the ddist-fix-outgoing-tls-yaml-doc branch November 3, 2025 12:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Mistake in dnsdist online documentation

3 participants