net-dns 0.6.1 → 0.7.0

data/lib/net/dns/{names/names.rb → names.rb}

@@ -1,23 +1,19 @@
-module Net
-  module DNS 
-    
+module Net # :nodoc:
+  module DNS
+
     module Names
-      
-      # Argument Error for class Net::DNS::Names.
-      class ArgumentError < ArgumentError
-      end
-      
+
       # Base error class.
       class Error < StandardError
       end
-      
+
       # Generic Names Error.
       class ExpandError < Error
       end
-      
-      
+
+
       INT16SZ = 2
-      
+
       # Expand a compressed name in a DNS Packet object. Please
       # see RFC1025 for an explanation of how the compression
       # in DNS packets works, how may it be useful and how should
@@ -33,7 +29,7 @@ module Net
         while true
           raise ExpandError, "Offset is greater than packet lenght!" if packetlen < (offset+1)
           len = packet.unpack("@#{offset} C")[0]
-          
+
           if len == 0
             offset += 1
             break
@@ -54,16 +50,19 @@ module Net
             offset += len
           end
         end
-        return [name,offset] # name.chomp(".") if trailing dot has to be omitted
+        [name, offset] # name.chomp(".") if trailing dot has to be omitted
       end
-      
+
       def pack_name(name)
-        if name.size > 63
-          raise ArgumentError, "Label data cannot exceed 63 chars"
+        if name.size > 255 
+          raise ArgumentError, "Name may not exceed 255 chars"
         end
         arr = name.split(".")
         str = ""
         arr.each do |elem|
+          if elem.size > 63
+            raise ArgumentError, "Label may not exceed 63 chars"
+          end
           str += [elem.size,elem].pack("Ca*")
         end
         str += [0].pack("C")
@@ -105,7 +104,7 @@ module Net
         end
         return str,offset,names
       end
-      
+
       def valid?(name)
         if name =~ /^([a-z0-9]([-a-z0-9]*[a-z0-9])?\.)+((a[cdefgilmnoqrstuwxz]|aero|arpa)|(b[abdefghijmnorstvwyz]|biz)|(c[acdfghiklmnorsuvxyz]|cat|com|coop)|d[ejkmoz]|(e[ceghrstu]|edu)|f[ijkmor]|(g[abdefghilmnpqrstuwy]|gov)|h[kmnrtu]|(i[delmnoqrst]|info|int)|(j[emop]|jobs)|k[eghimnprwyz]|l[abcikrstuvy]|(m[acdghklmnopqrstuvwxyz]|mil|mobi|museum)|(n[acefgilopruz]|name|net)|(om|org)|(p[aefghklmnrstwy]|pro)|qa|r[eouw]|s[abcdeghijklmnortvyz]|(t[cdfghjklmnoprtvwz]|travel)|u[agkmsyz]|v[aceginu]|w[fs]|y[etu]|z[amw])$/i
           return name
@@ -113,7 +112,7 @@ module Net
           raise ArgumentError, "Invalid FQDN: #{name}"
         end
       end
-      
+
     end
   end
-end
+end

data/lib/net/dns/packet.rb

@@ -1,36 +1,28 @@
 require 'logger'
-require 'net/dns/names/names'
-require 'net/dns'
+require 'net/dns/names'
 require 'net/dns/header'
 require 'net/dns/question'
 require 'net/dns/rr'
 
-module Net # :nodoc:
-  module DNS 
-    
-    # =Name
-    #
-    # Net::DNS::Packet - DNS packet object class
+module Net
+  module DNS
+
     #
-    # =Synopsis
-    # 
-    #   require 'net/dns/packet'
+    # = Net::DNS::Packet
     #
-    # =Description
-    # 
     # The Net::DNS::Packet class represents an entire DNS packet,
-    # divided in his main section: 
-    # 
+    # divided in his main section:
+    #
     # * Header (instance of Net::DNS::Header)
     # * Question (array of Net::DNS::Question objects)
-    # * Answer, Authority, Additional (each formed by an array of Net::DNS::RR 
+    # * Answer, Authority, Additional (each formed by an array of Net::DNS::RR
     #   objects)
     #
     # You can use this class whenever you need to create a DNS packet, whether
     # in an user application, in a resolver instance (have a look, for instance,
-    # at the Net::DNS::Resolver#send method) or for a nameserver.
+    # at the <tt>Net::DNS::Resolver#send</tt> method) or for a nameserver.
     #
-    # Some example:
+    # For example:
     #
     #   # Create a packet
     #   packet = Net::DNS::Packet.new("www.example.com")
@@ -39,7 +31,7 @@ module Net # :nodoc:
     #   # Getting packet binary data, suitable for network transmission
     #   data = packet.data
     #
-    # A packet object can be created from binary data too, like an 
+    # A packet object can be created from binary data too, like an
     # answer packet just received from a network stream:
     #
     #   packet = Net::DNS::Packet::parse(data)
@@ -48,13 +40,13 @@ module Net # :nodoc:
     #
     #   header = packet.header     # Instance of Net::DNS::Header class
     #   question = packet.question # Instance of Net::DNS::Question class
-    #   
+    #
     #   # Iterate over additional RRs
     #   packet.additional.each do |rr|
     #     puts "Got an #{rr.type} record"
     #   end
     #
-    # Some iterators have been written to easy the access of those RRs, 
+    # Some iterators have been written to easy the access of those RRs,
     # which are often the most important. So instead of doing:
     #
     #   packet.answer.each do |rr|
@@ -71,10 +63,10 @@ module Net # :nodoc:
     #
     # Be sure you don't miss all the iterators in the class documentation.
     #
-    # =Logging facility
-    # 
+    # == Logging facility
+    #
     # As Net::DNS::Resolver class, Net::DNS::Packet class has its own logging
-    # facility too. It work in the same way the other one do, so you can 
+    # facility too. It work in the same way the other one do, so you can
     # maybe want to override it or change the file descriptor.
     #
     #   packet = Net::DNS::Packet.new("www.example.com")
@@ -83,49 +75,29 @@ module Net # :nodoc:
     #   # or even
     #   packet.logger = Logger.new("/tmp/packet.log")
     #
-    # If the Net::DNS::Packet class is directly instantiated by the Net::DNS::Resolver
+    # If the <tt>Net::DNS::Packet</tt> class is directly instantiated by the <tt>Net::DNS::Resolver</tt>
     # class, like the great majority of the time, it will use the same logger facility.
     #
-    # Logger level will be set to Logger::Debug if $DEBUG variable is set.
-    #
-    # =Error classes
-    #
-    # Some error classes has been defined for the Net::DNS::Packet class,
-    # which are listed here to keep a light and browsable main documentation.
-    # We have:
-    #
-    # ArgumentError::   Argument Error for class Net::DNS::Packet
-    # PacketError::     Generic Packet Error
-    #
-    # =Copyright
-    # 
-    # Copyright (c) 2006 Marco Ceresa
-    #
-    # All rights reserved. This program is free software; you may redistribute 
-    # it and/or modify it under the same terms as Ruby itself.
+    # Logger level will be set to <tt>Logger::Debug</tt> if <tt>$DEBUG</tt> variable is set.
     #
     class Packet
       include Names
-      
-      # Argument Error for class Net::DNS::Packet.
-      class ArgumentError < ArgumentError
-      end
-      
+
       # Base error class.
       class Error < StandardError
       end
-      
+
       # Generic Packet Error.
       class PacketError < Error
       end
-      
-      
+
+
       attr_reader :header, :question, :answer, :authority, :additional
       attr_reader :answerfrom, :answersize
-      
-      # Create a new instance of Net::DNS::Packet class. Arguments are the
-      # canonical name of the resourse, an optional type field and an optional
-      # class field. The record type and class can be omitted; they default 
+
+      # Creates a new instance of <tt>Net::DNS::Packet</tt> class. Arguments are the
+      # canonical name of the resource, an optional type field and an optional
+      # class field. The record type and class can be omitted; they default
       # to +A+ and +IN+.
       #
       #   packet = Net::DNS::Packet.new("www.example.com")
@@ -133,8 +105,7 @@ module Net # :nodoc:
       #   packet = Net::DNS::Packet.new("example.com", Net::DNS::TXT, Net::DNS::CH)
       #
       # This class no longer instantiate object from binary data coming from
-      # network streams. Please use Net::DNS::Packet.new_from_data instead.
-      # 
+      # network streams. Please use <tt>Net::DNS::Packet.parse</tt> instead.
       def initialize(name, type = Net::DNS::A, cls = Net::DNS::IN)
         @header = Net::DNS::Header.new(:qdCount => 1)
         @question = [Net::DNS::Question.new(name, type, cls)]
@@ -145,33 +116,13 @@ module Net # :nodoc:
         @logger.level = $DEBUG ? Logger::DEBUG : Logger::WARN
       end
 
-      # Create a new instance of Net::DNS::Packet class from binary data, taken
-      # out by a network stream. For example:
-      #
-      #   # udp_socket is an UDPSocket waiting for a response
-      #   ans = udp_socket.recvfrom(1500)
-      #   packet = Net::DNS::Packet::parse(ans)
-      #
-      # An optional +from+ argument can be used to specify the information
-      # of the sender. If data is passed as is from a Socket#recvfrom call,
-      # the method will accept it.
-      #
-      # Be sure that your network data is clean from any UDP/TCP header, 
-      # expecially when using RAW sockets.
-      # 
-      def Packet.parse(*args)
-        o = allocate
-        o.send(:new_from_data, *args)
-        o
-      end
-      
 
       # Checks if the packet is a QUERY packet
       def query?
         @header.opCode == Net::DNS::Header::QUERY
       end
 
-      # Return the packet object in binary data, suitable
+      # Returns the packet object in binary data, suitable
       # for sending across a network stream.
       #
       #   packet_data = packet.data
@@ -181,7 +132,7 @@ module Net # :nodoc:
         qdcount=ancount=nscount=arcount=0
         data = @header.data
         headerlength = data.length
-        
+
         @question.each do |question|
           data += question.data
           qdcount += 1
@@ -198,7 +149,7 @@ module Net # :nodoc:
           data += rr.data#(data.length)
           arcount += 1
         end
-        
+
         @header.qdCount = qdcount
         @header.anCount = ancount
         @header.nsCount = nscount
@@ -207,13 +158,13 @@ module Net # :nodoc:
         @header.data + data[Net::DNS::HFIXEDSZ..data.size]
       end
 
-      # Same as Net::DNS::Packet#data, but implements name compression
+      # Same as <tt>Net::DNS::Packet#data</tt>, but implements name compression
       # (see RFC1025) for a considerable save of bytes.
       #
       #   packet = Net::DNS::Packet.new("www.example.com")
       #   puts "Size normal is #{packet.data.size} bytes"
       #   puts "Size compressed is #{packet.data_comp.size} bytes"
-      #   
+      #
       def data_comp
         offset = 0
         compnames = {}
@@ -227,46 +178,47 @@ module Net # :nodoc:
           compnames.update(names)
           qdcount += 1
         end
-        
+
         @answer.each do |rr|
           str,offset,names = rr.data(offset,compnames)
           data += str
           compnames.update(names)
           ancount += 1
         end
-        
+
         @authority.each do |rr|
           str,offset,names = rr.data(offset,compnames)
           data += str
           compnames.update(names)
           nscount += 1
         end
-        
+
         @additional.each do |rr|
           str,offset,names = rr.data(offset,compnames)
           data += str
           compnames.update(names)
           arcount += 1
         end
-        
+
         @header.qdCount = qdcount
         @header.anCount = ancount
         @header.nsCount = nscount
         @header.arCount = arcount
-        
+
         @header.data + data[Net::DNS::HFIXEDSZ..data.size]
       end
-      
-      # Inspect method
+
+      # Returns a string containing a human-readable representation
+      # of this <tt>Net::DNS::Packet</tt> instance.
       def inspect
         retval = ""
         if @answerfrom != "0.0.0.0:0" and @answerfrom
           retval += ";; Answer received from #@answerfrom (#{@answersize} bytes)\n;;\n"
         end
-        
+
         retval += ";; HEADER SECTION\n"
         retval += @header.inspect
-        
+
         retval += "\n"
         section = (@header.opCode == "UPDATE") ? "ZONE" : "QUESTION"
         retval += ";; #{section} SECTION (#{@header.qdCount} record#{@header.qdCount == 1 ? '' : 's'}):\n"
@@ -291,7 +243,7 @@ module Net # :nodoc:
             retval += rr.inspect + "\n"
           end
         end
-        
+
         unless @additional.size == 0
           retval += "\n"
           retval += ";; ADDITIONAL SECTION (#{@header.arCount} record#{@header.arCount == 1 ? '' : 's'}):\n"
@@ -299,20 +251,18 @@ module Net # :nodoc:
             retval += rr.inspect + "\n"
           end
         end
-        
+
         retval
       end
-      alias_method :to_s, :inspect 
-      
-      # Wrapper to Header#truncated?
-      #
+      alias_method :to_s, :inspect
+
+      # Delegates to <tt>Net::DNS::Header#truncated?</tt>.
       def truncated?
         @header.truncated?
       end
-            
-      # Assing a Net::DNS::Header object to a Net::DNS::Packet 
-      # instance.
-      #
+
+      # Assigns a <tt>Net::DNS::Header</tt> <tt>object</tt>
+      # to this <tt>Net::DNS::Packet</tt> instance.
       def header=(object)
         if object.kind_of? Net::DNS::Header
           @header = object
@@ -320,10 +270,9 @@ module Net # :nodoc:
           raise ArgumentError, "Argument must be a Net::DNS::Header object"
         end
       end
-      
-      # Assign a Net::DNS::Question object, or an array of 
-      # Questions objects, to a Net::DNS::Packet instance.
-      #
+
+      # Assigns a <tt>Net::DNS::Question</tt> <tt>object</tt>
+      # to this <tt>Net::DNS::Packet</tt> instance.
       def question=(object)
         case object
         when Array
@@ -339,250 +288,276 @@ module Net # :nodoc:
         end
       end
 
-      # Assign a Net::DNS::RR object, or an array of 
-      # RR objects, to a Net::DNS::Packet instance answer 
-      # section.
-      #
+      # Assigns one or an array of <tt>Net::DNS::RR</tt> <tt>object</tt>s
+      # to the answer section of this <tt>Net::DNS::Packet</tt> instance.
       def answer=(object)
         case object
-        when Array
-          if object.all? {|x| x.kind_of? Net::DNS::RR}
-            @answer = object
+          when Array
+            if object.all? {|x| x.kind_of? Net::DNS::RR}
+              @answer = object
+            else
+              raise ArgumentError, "Some of the elements is not an Net::DNS::RR object"
+            end
+          when Net::DNS::RR
+            @answer = [object]
           else
-            raise ArgumentError, "Some of the elements is not an Net::DNS::RR object"
-          end
-        when Net::DNS::RR
-          @answer = [object]
-        else
-          raise ArgumentError, "Invalid argument, not a RR object nor an array of objects"
+            raise ArgumentError, "Invalid argument, not a RR object nor an array of objects"
         end
       end
 
-      # Assign a Net::DNS::RR object, or an array of 
-      # RR objects, to a Net::DNS::Packet instance additional 
-      # section.
-      #
+      # Assigns one or an array of <tt>Net::DNS::RR</tt> <tt>object</tt>s
+      # to the additional section of this <tt>Net::DNS::Packet</tt> instance.
       def additional=(object)
         case object
-        when Array
-          if object.all? {|x| x.kind_of? Net::DNS::RR}
-            @additional = object
+          when Array
+            if object.all? {|x| x.kind_of? Net::DNS::RR}
+              @additional = object
+            else
+              raise ArgumentError, "Some of the elements is not an Net::DNS::RR object"
+            end
+          when Net::DNS::RR
+            @additional = [object]
           else
-            raise ArgumentError, "Some of the elements is not an Net::DNS::RR object"
-          end
-        when Net::DNS::RR
-          @additional = [object]
-        else
-          raise ArgumentError, "Invalid argument, not a RR object nor an array of objects"
+            raise ArgumentError, "Invalid argument, not a RR object nor an array of objects"
         end
       end
 
-      # Assign a Net::DNS::RR object, or an array of 
-      # RR objects, to a Net::DNS::Packet instance authority 
-      # section.
-      #
+      # Assigns one or an array of <tt>Net::DNS::RR</tt> <tt>object</tt>s
+      # to the authority section of this <tt>Net::DNS::Packet</tt> instance.
       def authority=(object)
         case object
-        when Array
-          if object.all? {|x| x.kind_of? Net::DNS::RR}
-            @authority = object
+          when Array
+            if object.all? {|x| x.kind_of? Net::DNS::RR}
+              @authority = object
+            else
+              raise ArgumentError, "Some of the elements is not an Net::DNS::RR object"
+            end
+          when Net::DNS::RR
+            @authority = [object]
           else
-            raise ArgumentError, "Some of the elements is not an Net::DNS::RR object"
-          end
-        when Net::DNS::RR
-          @authority = [object]
-        else
-          raise ArgumentError, "Invalid argument, not a RR object nor an array of objects"
+            raise ArgumentError, "Invalid argument, not a RR object nor an array of objects"
         end
       end
-      
-      # Iterate for every address in the +answer+ section of a 
-      # Net::DNS::Packet object.
+
+      # Iterates every address in the +answer+ section
+      # of this <tt>Net::DNS::Packet</tt> instance.
       #
       #   packet.each_address do |ip|
       #     ping ip.to_s
       #   end
       #
-      # As you can see in the documentation for Net::DNS::RR::A class,
-      # the address returned is an instance of IPAddr class. 
-      #
-      def each_address
+      # As you can see in the documentation for the <tt>Net::DNS::RR::A</tt> class,
+      # the address returned is an instance of <tt>IPAddr</tt> class.
+      def each_address(&block)
         @answer.each do |elem|
           next unless elem.class == Net::DNS::RR::A
           yield elem.address
         end
       end
-      
-      # Iterate for every nameserver in the +answer+ section of a 
-      # Net::DNS::Packet object.
+
+      # Iterates every nameserver in the +answer+ section
+      # of this <tt>Net::DNS::Packet</tt> instance.
       #
       #   packet.each_nameserver do |ns|
       #     puts "Nameserver found: #{ns}"
       #   end
       #
-      def each_nameserver
+      def each_nameserver(&block)
         @answer.each do |elem|
           next unless elem.class == Net::DNS::RR::NS
           yield elem.nsdname
         end
       end
-      
-      # Iterate for every exchange record in the +answer+ section 
-      # of a Net::DNS::Packet object.
+
+      # Iterates every exchange record in the +answer+ section
+      # of this <tt>Net::DNS::Packet</tt> instance.
       #
       #   packet.each_mx do |pref,name|
       #     puts "Mail exchange #{name} has preference #{pref}"
       #   end
       #
-      def each_mx
+      def each_mx(&block)
         @answer.each do |elem|
           next unless elem.class == Net::DNS::RR::MX
-          yield elem.preference,elem.exchange
+          yield elem.preference, elem.exchange
         end
       end
-      
-      # Iterate for every canonical name in the +answer+ section 
-      # of a Net::DNS::Packet object.
+
+      # Iterates every canonical name in the +answer+ section
+      # of this <tt>Net::DNS::Packet</tt> instance.
       #
       #   packet.each_cname do |cname|
       #     puts "Canonical name: #{cname}"
       #   end
       #
-      def each_cname
+      def each_cname(&block)
         @answer.each do |elem|
           next unless elem.class == Net::DNS::RR::CNAME
           yield elem.cname
         end
       end
-      
-      # Iterate for every pointer in the +answer+ section of a 
-      # Net::DNS::Packet object.
+
+      # Iterates every pointer in the +answer+ section
+      # of this <tt>Net::DNS::Packet</tt> instance.
       #
       #   packet.each_ptr do |ptr|
       #     puts "Pointer for resource: #{ptr}"
       #   end
       #
-      def each_ptr
+      def each_ptr(&block)
         @answer.each do |elem|
           next unless elem.class == Net::DNS::RR::PTR
           yield elem.ptrdname
         end
       end
 
-      # Returns the packet size in bytes
+      # Returns the packet size in bytes.
       #
-      #   Resolver("www.google.com") do |packet| 
+      #   Resolver("www.google.com") do |packet|
       #     puts packet.size + " bytes"}
       #   end
-      #     #=> 484 bytes
+      #   # => 484 bytes
       #
       def size
         data.size
       end
 
-      # Checks whether a query has returned a NXDOMAIN error,
-      # meaning the domain name queried doesn't exists.
+      # Checks whether the query returned a NXDOMAIN error,
+      # meaning the queried domain name doesn't exist.
       #
       #   %w[a.com google.com ibm.com d.com].each do |domain|
       #     response = Net::DNS::Resolver.new.send(domain)
       #     puts "#{domain} doesn't exist" if response.nxdomain?
       #   end
-      #     #=> a.com doesn't exist
-      #     #=> d.com doesn't exist
+      #   # => a.com doesn't exist
+      #   # => d.com doesn't exist
       #
       def nxdomain?
         header.rCode.code == Net::DNS::Header::RCode::NAME
       end
-      
+
+
+      # Creates a new instance of <tt>Net::DNS::Packet</tt> class from binary data,
+      # taken out from a network stream. For example:
+      #
+      #   # udp_socket is an UDPSocket waiting for a response
+      #   ans = udp_socket.recvfrom(1500)
+      #   packet = Net::DNS::Packet::parse(ans)
+      #
+      # An optional +from+ argument can be used to specify the information
+      # of the sender. If data is passed as is from a Socket#recvfrom call,
+      # the method will accept it.
+      #
+      # Be sure that your network data is clean from any UDP/TCP header,
+      # especially when using RAW sockets.
+      #
+      def self.parse(*args)
+        o = allocate
+        o.send(:new_from_data, *args)
+        o
+      end
+
       private
 
-      # New packet from binary data
-      def new_from_data(data, from = nil)
-        unless from
-          if data.kind_of? Array
-            data,from = data
-          else
-            from = [0,0,"0.0.0.0","unknown"]
+        # New packet from binary data
+        def new_from_data(data, from = nil)
+          unless from
+            if data.kind_of? Array
+              data, from = data
+            else
+              from = [0, 0, "0.0.0.0", "unknown"]
+            end
           end
-        end
-          
-        @answerfrom = from[2] + ":" + from[1].to_s
-        @answersize = data.size
-        @logger = Logger.new $stdout
-        @logger.level = $DEBUG ? Logger::DEBUG : Logger::WARN
-        
-        #------------------------------------------------------------
-        # Header section
-        #------------------------------------------------------------
-        offset = Net::DNS::HFIXEDSZ 
-        @header = Net::DNS::Header.parse(data[0..offset-1])
-
-        @logger.debug ";; HEADER SECTION"
-        @logger.debug @header.inspect
-
-        #------------------------------------------------------------
-        # Question section
-        #------------------------------------------------------------
-        section = @header.opCode == "UPDATE" ? "ZONE" : "QUESTION"
-        @logger.debug ";; #{section} SECTION (#{@header.qdCount} record#{@header.qdCount == 1 ? '': 's'})"
-
-        @question = []
-        @header.qdCount.times do
-          qobj,offset = parse_question(data,offset)
-          @question << qobj
-          @logger.debug ";; #{qobj.inspect}"
-        end
 
-        #------------------------------------------------------------
-        # Answer/prerequisite section
-        #------------------------------------------------------------
-        section = @header.opCode == "UPDATE" ? "PREREQUISITE" : "ANSWER"
-        @logger.debug ";; #{section} SECTION (#{@header.qdCount} record#{@header.qdCount == 1 ? '': 's'})"
-        
-        @answer = []
-        @header.anCount.times do
-          rrobj,offset = Net::DNS::RR.parse_packet(data,offset)
-          @answer << rrobj
-          @logger.debug rrobj.inspect
-        end
+          @answerfrom = from[2] + ":" + from[1].to_s
+          @answersize = data.size
+          @logger = Logger.new $stdout
+          @logger.level = $DEBUG ? Logger::DEBUG : Logger::WARN
+
+          #------------------------------------------------------------
+          # Header section
+          #------------------------------------------------------------
+          offset = Net::DNS::HFIXEDSZ
+          @header = Net::DNS::Header.parse(data[0..offset-1])
+
+          @logger.debug ";; HEADER SECTION"
+          @logger.debug @header.inspect
+
+          #------------------------------------------------------------
+          # Question section
+          #------------------------------------------------------------
+          section = @header.opCode == "UPDATE" ? "ZONE" : "QUESTION"
+          @logger.debug ";; #{section} SECTION (#{@header.qdCount} record#{@header.qdCount == 1 ? '': 's'})"
+
+          @question = []
+          @header.qdCount.times do
+            qobj,offset = parse_question(data,offset)
+            @question << qobj
+            @logger.debug ";; #{qobj.inspect}"
+          end
+
+          #------------------------------------------------------------
+          # Answer/prerequisite section
+          #------------------------------------------------------------
+          section = @header.opCode == "UPDATE" ? "PREREQUISITE" : "ANSWER"
+          @logger.debug ";; #{section} SECTION (#{@header.qdCount} record#{@header.qdCount == 1 ? '': 's'})"
+
+          @answer = []
+          @header.anCount.times do
+            begin
+              rrobj,offset = Net::DNS::RR.parse_packet(data,offset)
+              @answer << rrobj
+              @logger.debug rrobj.inspect
+            rescue NameError => e
+              warn "Net::DNS unsupported record type: #{e.message}"
+            end
+          end
+
+          #------------------------------------------------------------
+          # Authority/update section
+          #------------------------------------------------------------
+          section = @header.opCode == "UPDATE" ? "UPDATE" : "AUTHORITY"
+          @logger.debug ";; #{section} SECTION (#{@header.nsCount} record#{@header.nsCount == 1 ? '': 's'})"
+
+          @authority = []
+          @header.nsCount.times do
+            begin
+              rrobj,offset = Net::DNS::RR.parse_packet(data,offset)
+              @authority << rrobj
+              @logger.debug rrobj.inspect
+            rescue NameError => e
+              warn "Net::DNS unsupported record type: #{e.message}"
+            end
+          end
+
+          #------------------------------------------------------------
+          # Additional section
+          #------------------------------------------------------------
+          @logger.debug ";; ADDITIONAL SECTION (#{@header.arCount} record#{@header.arCount == 1 ? '': 's'})"
+
+          @additional = []
+          @header.arCount.times do
+            begin
+              rrobj,offset = Net::DNS::RR.parse_packet(data,offset)
+              @additional << rrobj
+              @logger.debug rrobj.inspect
+            rescue NameError => e
+              warn "Net::DNS supported record type: #{e.message}"
+            end
+          end
 
-        #------------------------------------------------------------
-        # Authority/update section
-        #------------------------------------------------------------
-        section = @header.opCode == "UPDATE" ? "UPDATE" : "AUTHORITY"
-        @logger.debug ";; #{section} SECTION (#{@header.nsCount} record#{@header.nsCount == 1 ? '': 's'})"
-        
-        @authority = []
-        @header.nsCount.times do
-          rrobj,offset = Net::DNS::RR.parse_packet(data,offset)
-          @authority << rrobj
-          @logger.debug rrobj.inspect          
         end
-        
-        #------------------------------------------------------------
-        # Additional section
-        #------------------------------------------------------------
-        @logger.debug ";; ADDITIONAL SECTION (#{@header.arCount} record#{@header.arCount == 1 ? '': 's'})"    
-        
-        @additional = []
-        @header.arCount.times do
-          rrobj,offset = Net::DNS::RR.parse_packet(data,offset)
-          @additional << rrobj
-          @logger.debug rrobj.inspect
+
+
+        # Parse question section
+        def parse_question(data,offset)
+          size = (dn_expand(data, offset)[1] - offset) + (2 * Net::DNS::INT16SZ)
+          return [Net::DNS::Question.parse(data[offset, size]), offset + size]
+        rescue StandardError => e
+          raise PacketError, "Caught exception, maybe packet malformed => #{e.message}"
         end
-        
-      end # new_from_data
-      
-      
-      # Parse question section
-      def parse_question(data,offset)
-        size = (dn_expand(data,offset)[1]-offset) + 2*Net::DNS::INT16SZ
-        return [Net::DNS::Question.parse(data[offset,size]), offset+size]
-      rescue StandardError => e
-        raise PacketError, "Caught exception, maybe packet malformed => #{e.message}"
-      end
 
     end
-    
+
   end
 end