root zone fixes in core and gsql backends #3637

Merged
merged 11 commits into from Apr 7, 2016

Projects

None yet

4 participants

@mind04
Contributor
mind04 commented Mar 30, 2016

Core and gsql backends are able to serve/AXFR a root/tld zone
Bind backend still need some love (in a separate pull)

@Habbie
Member
Habbie commented Apr 7, 2016

This branch has conflicts

@Habbie Habbie commented on an outdated diff Apr 7, 2016
...ession-tests.rootzone/tests/direct-ns/expected_result
@@ -0,0 +1,29 @@
+1 net. IN NS 172800 a.gtld-servers.net.
+1 net. IN NS 172800 b.gtld-servers.net.
+1 net. IN NS 172800 c.gtld-servers.net.
+1 net. IN NS 172800 d.gtld-servers.net.
+1 net. IN NS 172800 e.gtld-servers.net.
+1 net. IN NS 172800 f.gtld-servers.net.
+1 net. IN NS 172800 g.gtld-servers.net.
+1 net. IN NS 172800 h.gtld-servers.net.
+1 net. IN NS 172800 i.gtld-servers.net.
+1 net. IN NS 172800 j.gtld-servers.net.
+1 net. IN NS 172800 k.gtld-servers.net.
+1 net. IN NS 172800 l.gtld-servers.net.
+1 net. IN NS 172800 m.gtld-servers.net.
@Habbie
Habbie Apr 7, 2016 Member

If your ROOT file only has two NSes in each set, why are we getting 13 here?

@Habbie
Habbie Apr 7, 2016 Member

Nevermind, later commit fixes this!

@Habbie
Member
Habbie commented Apr 7, 2016

This is awesome work from both of you! Happy to merge this when the conflicts are gone.

@cmouse
Contributor
cmouse commented Apr 7, 2016

" AXFR-out rectify: don't set odername for empty non-terminals only der… … ", typo fix if you are rebasing

@pieterlexis pieterlexis merged commit 6ded341 into PowerDNS:master Apr 7, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@mind04 mind04 deleted the mind04:root branch Apr 7, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment