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.

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

Zone diff support is incomplete/incorrect. #405

Open
ximon18 opened this issue Oct 7, 2024 · 0 comments
Open

Zone diff support is incomplete/incorrect. #405

ximon18 opened this issue Oct 7, 2024 · 0 comments
Labels
bug Something isn't working unstable feature

Comments

@ximon18
Copy link
Member

ximon18 commented Oct 7, 2024

When a zone is updated via the WritableZoneNode::update_rrset() method a diff of the effect can optionally be created.

No such support for creating diffs is implemented however when the "special" make_xxx() methods are used to update a zone, e.g. make_zone_cut() or make_cname().

Such updates will not appear in the created diff.

Additionally, changing the type of a node from "special" to "normal" by calling WritableZoneNode::update_rrset() on a node which was previously created using one of the make_xxx() methods will cause an incorrect diff to be recorded showing that the node changed from empty to having a "normal" RRset, while actually it went from having a "special" RRset to a "normal" RRset.

@ximon18 ximon18 added bug Something isn't working unstable feature labels Oct 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working unstable feature
Projects
None yet
Development

No branches or pull requests

1 participant