Changeset 2197

Timestamp:

Feb 27, 2009, 2:41:51 AM (16 years ago)

Author:

quentin

Message:

Reverse-resolution support in invirt-dns

Location:

trunk/packages

Files:

• invirt-dns/debian/changelog (modified) (1 diff)
• invirt-dns/invirt-dns (modified) (1 diff)
• xvm-devconfig/debian/changelog (modified) (1 diff)
• xvm-devconfig/master.yaml (modified) (1 diff)

trunk/packages/invirt-dns/debian/changelog

@@ +1 @@
+invirt-dns (0.0.10) unstable; urgency=low
+
+  * Add support for resolving PTR records in the in-addr.arpa zone
+
+ -- Quentin Smith <quentin@mit.edu>  Fri, 27 Feb 2009 02:39:25 -0500
+
 invirt-dns (0.0.9) unstable; urgency=low
 

trunk/packages/invirt-dns/invirt-dns

@@ -94 +94 @@
                                                 ttl, self.soa, auth=True))
             else: # Request for a subdomain.
-                value = invirt.database.NIC.query.filter_by(hostname=host).first()
-                if value:
-                    ip = value.ip
-                else:
-                    value = invirt.database.Machine.query().filter_by(name=host).first()
+                if name.endswith(".in-addr.arpa"): # Reverse resolution here
+                    if type in (dns.PTR, dns.ALL_RECORDS):
+                        ip = '.'.join(reversed(name.split('.')[:-2]))
+                        value = invirt.database.NIC.query.filter_by(ip=ip).first()
+                        if value and value.hostname:
+                            hostname = value.hostname
+                            if '.' not in hostname:
+                                hostname = hostname + "." + config.dns.domains[0]
+                            record = dns.Record_PTR(hostname, ttl)
+                            results.append(dns.RRHeader(name, dns.PTR, dns.IN,
+                                                        ttl, record, auth=True))
+                        else: # IP address doesn't point to an active host
+                            return defer.fail(failure.Failure(dns.AuthoritativeDomainError(name)))
+                    # FIXME: Should only return success with no records if the name actually exists
+                else: # Forward resolution here
+                    value = invirt.database.NIC.query.filter_by(hostname=host).first()
                     if value:
-                        ip = value.nics[0].ip
+                        ip = value.ip
                     else:
+                        value = invirt.database.Machine.query().filter_by(name=host).first()
+                        if value:
+                            ip = value.nics[0].ip
+                        else:
+                            return defer.fail(failure.Failure(dns.AuthoritativeDomainError(name)))
+                
+                    if ip is None:
                         return defer.fail(failure.Failure(dns.AuthoritativeDomainError(name)))
-                
-                if ip is None:
-                    return defer.fail(failure.Failure(dns.AuthoritativeDomainError(name)))
-
-                if type in (dns.A, dns.ALL_RECORDS):
-                    record = dns.Record_A(ip, ttl)
-                    results.append(dns.RRHeader(name, dns.A, dns.IN, 
-                                                ttl, record, auth=True))
-                elif type == dns.SOA:
-                    results.append(dns.RRHeader(domain, dns.SOA, dns.IN,
-                                                ttl, self.soa, auth=True))
+
+                    if type in (dns.A, dns.ALL_RECORDS):
+                        record = dns.Record_A(ip, ttl)
+                        results.append(dns.RRHeader(name, dns.A, dns.IN, 
+                                                    ttl, record, auth=True))
+                    elif type == dns.SOA:
+                        results.append(dns.RRHeader(domain, dns.SOA, dns.IN,
+                                                    ttl, self.soa, auth=True))
             if len(results) == 0:
                 authority = []

trunk/packages/xvm-devconfig/debian/changelog

@@ -1 +1 @@
 xvm-devconfig (0.29) unstable; urgency=low
 
-  * Act authoritative for 2.181.18.in-addr.arpa
+  * Act authoritative for {1,2}.181.18.in-addr.arpa
 
  -- Quentin Smith <quentin@mit.edu>  Fri, 27 Feb 2009 02:20:58 -0500

trunk/packages/xvm-devconfig/master.yaml

@@ -37 +37 @@
   - dev.xvm.mit.edu
   - 2.181.18.in-addr.arpa
+  - 1.181.18.in-addr.arpa
  nameservers:
   - hostname: xvm-dev.mit.edu