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

Import of existing bind zone doesn't handle AAAA records #78

Closed
bigwillystyle42 opened this Issue Mar 27, 2014 · 2 comments

Comments

Projects
None yet
2 participants
@bigwillystyle42
Contributor

bigwillystyle42 commented Mar 27, 2014

Hi,

I'm trying to import an existing bind zone with AAAA records and I'm seeing two broken behaviors:

  1. AAAA records after an A record are treated as duplicates of the A record
  2. AAAA records before any A records are dropped

This zone file demonstrates both issues:

$TTL   86400
@       IN      SOA     ns1.example.com. root.example.com. (
      2013042516    ; Serial
      28800         ; Refresh
      7200      ; Retry
      1209600   ; Expire
      86400)        ; Negative Cache TTL
;
@       IN      NS        ns1.example.com.
@       IN      NS        ns2.example.com.
@       IN      NS        ns3.example.com.
@       IN      NS        ns4.example.com.
@   IN  AAAA        2600:3c03::f03c:91ff:fedf:ef4d
@   IN  A       96.126.105.17
@   IN  MX  10  mail.example.com.
@ IN A 1.2.3.4
blah    IN  AAAA        2600:3c03::f03c:91ff:fedf:ef4d

@WillyXJ WillyXJ added bug labels Apr 3, 2014

@WillyXJ WillyXJ added this to the 1.2 release milestone Apr 3, 2014

@WillyXJ

This comment has been minimized.

Owner

WillyXJ commented Apr 3, 2014

I agree that the zone importer does not handle AAAA records very well. This will probably take more work than a simple bug fix so I've targeted it for v1.2.

WillyXJ added a commit that referenced this issue Apr 7, 2014

@WillyXJ

This comment has been minimized.

Owner

WillyXJ commented Apr 7, 2014

Changed merged in.

@WillyXJ WillyXJ closed this Apr 7, 2014

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