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.

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

Jump to bottom

Finish CDS/CDNSKEY implementation #114

Closed
vlevigneron opened this issue May 10, 2021 · 2 comments · Fixed by #199
Closed

Finish CDS/CDNSKEY implementation #114

vlevigneron opened this issue May 10, 2021 · 2 comments · Fixed by #199
Assignees
@tgreenx
Labels
S-PRforIssue Status: There is a PR that is meant to resolve the issue T-Feature Type: New feature in software or test case description
Milestone
v2024.2

Comments

@vlevigneron
Copy link
Contributor

  • Specific methods for CDS/CDNSKEY ressource records need to be implemented.
@matsduf matsduf mentioned this issue May 20, 2021
Open
@matsduf matsduf added this to the v2021.2 milestone May 27, 2021
@matsduf matsduf modified the milestones: v2021.2, v2022.1 Dec 8, 2021
@matsduf matsduf modified the milestones: v2022.1, v2022.2 May 21, 2022
@ghost
Copy link

ghost commented Oct 6, 2022

Since the CDS (resp CDNSKEY) resource record format is identical to the DS record (resp DNSKEY record), we could make the Zonemaster::LDNS::RR::CDS (resp Zonemaster::LDNS::RR::CDNSKEY) a subclass of Zonemaster::LDNS::RR::DS (resp Zonemaster::LDNS::RR::DNSKEY).

@ghost ghost mentioned this issue Oct 6, 2022
Closed
@matsduf matsduf assigned ghost Oct 6, 2022
@matsduf matsduf modified the milestones: v2022.2, v2023.1 Dec 20, 2022
@matsduf matsduf modified the milestones: v2023.1, v2023.2 Jun 26, 2023
@matsduf matsduf modified the milestones: v2023.2, v2024.1 Mar 19, 2024
@matsduf matsduf modified the milestones: v2024.1, v2024.2 Jul 3, 2024
@tgreenx tgreenx added the T-Feature Type: New feature in software or test case description label Aug 19, 2024
@tgreenx tgreenx self-assigned this Aug 20, 2024
@tgreenx tgreenx mentioned this issue Aug 20, 2024
Merged
@tgreenx tgreenx linked a pull request Aug 20, 2024 that will close this issue
Extend CDS/CDNSKEY support #199
Merged
@tgreenx tgreenx added the S-PRforIssue Status: There is a PR that is meant to resolve the issue label Aug 20, 2024
@tgreenx
Copy link
Contributor

tgreenx commented Sep 12, 2024

Solved by #199.

Note that for some methods to work properly for these types of records, an update in LDNS will be required (version 1.8.5 as it appears). See NLnetLabs/ldns#246, NLnetLabs/ldns#247, NLnetLabs/ldns#248.

@tgreenx tgreenx closed this as completed Sep 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tgreenx tgreenx

Labels
S-PRforIssue Status: There is a PR that is meant to resolve the issue T-Feature Type: New feature in software or test case description
Projects
None yet
Milestone
v2024.2
Development

Successfully merging a pull request may close this issue.

Extend CDS/CDNSKEY support tgreenx/zonemaster-ldns
3 participants
@matsduf @vlevigneron @tgreenx