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 Import should ensure singleton RR #280

Closed
lchanouha opened this Issue Jan 7, 2016 · 5 comments

Comments

Projects
None yet
2 participants
@lchanouha

lchanouha commented Jan 7, 2016

We can add for example both A and CNAME records for one subdomain.
This will trigger an error on bind and zone won't be reloaded. And worse if you restart the server your zone won't work anymore / no resolution.

A protection againts this kind of collpasing would be great to prevent some user errors.

@WillyXJ

This comment has been minimized.

Owner

WillyXJ commented Jan 7, 2016

Hello,

As of v2.1, this is in place per #208 - is this functionality not working for you?

@lchanouha

This comment has been minimized.

lchanouha commented Jan 7, 2016

Yes it didn't worked. I got 1 cname and 1 a records for same domain.
I imported my records via the zonefile import tool. Maybe there is no verifications on raw import ?

@WillyXJ

This comment has been minimized.

Owner

WillyXJ commented Jan 7, 2016

Ah, the zone import does not check for singleton RR. The import tool assumes the user is importing loadable zone files. This will need to be fixed.

@WillyXJ WillyXJ changed the title from Disallow different records type for same subdomain to Zone Import should ensure singleton RR Jan 7, 2016

@WillyXJ WillyXJ added this to the 2.x bug fixes milestone Jan 7, 2016

@WillyXJ

This comment has been minimized.

Owner

WillyXJ commented Mar 31, 2016

This fix will be in the next release.

@WillyXJ

This comment has been minimized.

Owner

WillyXJ commented Apr 25, 2016

v2.2 has been released and contains this fix.

@WillyXJ WillyXJ closed this Apr 25, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment