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.

Sign up for GitHub

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

WIP ZONEVERSION (RFC9660) support #14818

Draft

Habbie wants to merge 17 commits into PowerDNS:master from Habbie:zoneversion

Draft

WIP ZONEVERSION (RFC9660) support #14818

wider NOLINT on argv[i] access

GitHub Advanced Security / CodeQL succeeded Nov 14, 2024 in 4s

7 new alerts

New alerts in code changed by this pull request

7 notes

See annotations below for details.

View all branch alerts.

Annotations

Check notice on line 397 in pdns/dnspacket.cc

Code scanning / CodeQL

FIXME comment Note

FIXME comment: enum

Check notice on line 398 in pdns/dnspacket.cc

Code scanning / CodeQL

FIXME comment Note

FIXME comment: enum and wrong number

Check notice on line 34 in pdns/ednszoneversion.cc

Code scanning / CodeQL

FIXME comment Note

FIXME comment: they can be bigger

Check notice on line 41 in pdns/ednszoneversion.cc

Code scanning / CodeQL

Commented-out code Note

This comment appears to contain commented-out code.

Check notice on line 48 in pdns/ednszoneversion.cc

Code scanning / CodeQL

Commented-out code Note

This comment appears to contain commented-out code.

Check notice on line 50 in pdns/ednszoneversion.cc

Code scanning / CodeQL

FIXME comment Note

FIXME comment: this silently breaks on >256 bytes of version

Check notice on line 193 in pdns/sdig.cc

Code scanning / CodeQL

FIXME comment Note

FIXME comment: enum