gitlab-net-dns 0.10.0 → 0.12.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 04ce28e53e3b0986fb5a2dcde807ac07bc15ceb3bdd127abe31d125e4ae9e054
-  data.tar.gz: 119fcc2767ae448912fbadb013a82fdeff83fd68c06937fafdefecabbfea0d58
+  metadata.gz: 626fddeccebd199f0753f59a1f0da00bf17c923ab59863f6dcf4f1712e565aa9
+  data.tar.gz: 266c1c5e8b273c3b09305cca8a96127b4d46bfe9396ceb103ef3278edef54101
 SHA512:
-  metadata.gz: 0546f72172a83589a710a70cb8ff1e3caabab249cbd49a1ecf8751a802433d6ef6ff9b1667843772598248e7c0b155fd7836cf2aa1131da1be7fa1541bcf0a91
-  data.tar.gz: 469360aa30e2681543719347ce2a1dd488615df7f22eb7a6482bf2c58ce3e4b9d54fceab14de3de9b0a88e7394a3b2f76320877742c533f6a10e62a9fbb0660a
+  metadata.gz: f17432960ab586b0e2d74dc9d3d8cdad69a477c050f20139038155825b4dcd116d41fa8b423203310fd88cd8510f49086db89d2fee372219eb72fe5d39a56cf1
+  data.tar.gz: b357ac43e0927bccd981530f026f4af77f3a1da4d2263966621a9744a62781bf44f7438f383b404d3194f5a9f1f0f16ea490ac8116ad1299683aa5a408405af9

data/.gitlab-ci.yml

@@ -20,4 +20,4 @@ rspec:
     - bundle exec rake
   parallel:
     matrix:
-      - RUBY_VERSION: [ "2.7", "3.0", "3.1", "3.2" ]
+      - RUBY_VERSION: [ "3.0", "3.1", "3.2", "3.3", "3.4" ]

data/Gemfile

@@ -2,6 +2,7 @@ source "https://rubygems.org"
 
 gemspec
 
+gem "base64"
 gem "minitest"
 gem "minitest-reporters"
 gem "rspec"

data/gitlab-net-dns.gemspec

@@ -11,12 +11,13 @@ Gem::Specification.new do |s|
   s.description = "Net::DNS is a pure Ruby DNS library, with a clean OO interface and an extensible API."
   s.licenses    = ["Ruby"]
 
-  s.required_ruby_version = ">= 2.1"
+  s.required_ruby_version = ">= 3.0"
 
   s.require_paths    = ["lib"]
   s.files            = `git ls-files`.split("\n")
   s.test_files       = `git ls-files -- {test,spec,features}/*`.split("\n")
   s.extra_rdoc_files = %w[LICENSE.txt]
+  s.add_dependency "logger"
 
   s.add_development_dependency "mocha"
   s.add_development_dependency "rake"

data/lib/net/dns/rr/types.rb

@@ -84,6 +84,9 @@ module Net # :nodoc:
           'CAA' => 257, # RFC 6844
         }.freeze
 
+        # Pre-initialize the inverted hash to avoid runtime inversion
+        TYPES_BY_NUMBER = TYPES.invert.freeze
+
         # The default value when type is nil in Resource Records
         @@default = TYPES["A"]
 
@@ -107,7 +110,7 @@ module Net # :nodoc:
           when String
             TYPES.key?(type)
           when Integer
-            TYPES.invert.key?(type)
+            TYPES_BY_NUMBER.key?(type)
           else
             raise ArgumentError, "Wrong type class: #{type.class}"
           end
@@ -118,8 +121,8 @@ module Net # :nodoc:
         def self.to_str(type)
           case type
           when Integer
-            if TYPES.invert.key? type
-              TYPES.invert[type]
+            if TYPES_BY_NUMBER.key? type
+              TYPES_BY_NUMBER[type]
             else
               raise ArgumentError, "Unknown type number #{type}"
             end
@@ -148,7 +151,7 @@ module Net # :nodoc:
             new_from_num(type)
           when nil
             # default type, control with Types.default=
-            @str = TYPES.invert[@@default]
+            @str = TYPES_BY_NUMBER[@@default]
             @num = @@default
           else
             raise ArgumentError, "Wrong type class: #{type.class}"
@@ -197,11 +200,11 @@ module Net # :nodoc:
 
         # Contructor for numeric data type.
         def new_from_num(type)
-          if TYPES.invert.key? type
+          if TYPES_BY_NUMBER.key? type
             @num = type
-            @str = TYPES.invert[type]
+            @str = TYPES_BY_NUMBER[type]
           else
-            raise ArgumentError, "Unkown type number #{type}"
+            raise ArgumentError, "Unknown type number #{type}"
           end
         end
       end

data/lib/net/dns/rr.rb

@@ -298,7 +298,8 @@ module Net
         if self.class == Net::DNS::RR
           temp = dn_expand(data, offset)[1]
           type = Net::DNS::RR::Types.new data.unpack("@#{temp} n")[0]
-          (eval "Net::DNS::RR::#{type}").parse_packet(data, offset)
+          rr_class = Net::DNS::RR.const_get(type.to_s)
+          rr_class.parse_packet(data, offset)
         else
           @name, offset = dn_expand(data, offset)
           rrtype, cls, @ttl, @rdlength = data.unpack("@#{offset} n2 N n")

data/lib/net/dns/version.rb

@@ -3,6 +3,6 @@
 module Net
   module DNS
     # The current library version.
-    VERSION = "0.10.0"
+    VERSION = "0.12.0"
   end
 end

data/test/unit/rr_test.rb

@@ -1,4 +1,5 @@
 require 'test_helper'
+require 'base64'
 require 'net/dns/rr'
 
 class RRTest < Minitest::Test
@@ -124,4 +125,45 @@ class RRTest < Minitest::Test
       Net::DNS::RR.new("google.com. 10800 IM A")
     end
   end
+
+  def test_bad_type
+    assert_raises(ArgumentError) do
+      Net::DNS::RR.new(name: @rr_name,
+                       ttl: @ttl,
+                       cls: @cls,
+                       type: "BOGUS",
+                       address: @rdata)
+    end
+  end
+
+  def test_parse_packet_with_unsupported_type
+    # Create a mock binary packet with an unsupported RR type
+
+    # Start with a normal A record to get valid binary data
+    record = Net::DNS::RR.new("example.com. 3600 IN A 192.168.1.1")
+    binary_data = record.data
+
+    # Find where the name ends by looking for a null byte
+    name_end = binary_data.index("\x00") + 1
+
+    # Replace the type value (2 bytes) with 0xFACE (an undefined type)
+    modified_data = binary_data.dup
+    modified_data[name_end, 2] = [0xFACE].pack("n")
+
+    # The parse_packet should raise an error for unknown types
+    error = assert_raises(ArgumentError) do
+      Net::DNS::RR.parse(modified_data)
+    end
+
+    # Verify the error message mentions the unknown type
+    assert_match(/Unknown type number/, error.message)
+  end
+
+  def test_aaaa_packet
+    # A response of `dig google.com AAAA` query
+    encoded_payload = "PA+BgAABAAEADQAbBmdvb2dsZQNjb20AABwAAcAMABwAAQAAASwAECoAFFBA\nDggKAAAAAAAAIA7AEwACAAEAAAFMABQBYgxndGxkLXNlcnZlcnMDbmV0AMAT\nAAIAAQAAAUwABAFtwEbAEwACAAEAAAFMAAQBY8BGwBMAAgABAAABTAAEAWbA\nRsATAAIAAQAAAUwABAFnwEbAEwACAAEAAAFMAAQBa8BGwBMAAgABAAABTAAE\nAWXARsATAAIAAQAAAUwABAFqwEbAEwACAAEAAAFMAAQBZMBGwBMAAgABAAAB\nTAAEAWnARsATAAIAAQAAAUwABAFhwEbAEwACAAEAAAFMAAQBbMBGwBMAAgAB\nAAABTAAEAWjARsD0AAEAAQAAAUwABMAFBh7A9AAcAAEAAAFMABAgAQUDqD4A\nAAAAAAAAAgAwwEQAAQABAAABTAAEwCEOHsBEABwAAQAAAUwAECABBQMjHQAA\nAAAAAAACADDAdAABAAEAAAFMAATAGlwewHQAHAABAAABTAAQIAEFA4PrAAAA\nAAAAAAAAMMDUAAEAAQAAAUwABMAfUB7A1AAcAAEAAAFMABAgAQUAhW4AAAAA\nAAAAAAAwwLQAAQABAAABTAAEwAxeHsC0ABwAAQAAAUwAECABBQIcoQAAAAAA\nAAAAADDAhAABAAEAAAFMAATAIzMewIQAHAABAAABTAAQIAEFA9QUAAAAAAAA\nAAAAMMCUAAEAAQAAAUwABMAqXR7AlAAcAAEAAAFMABAgAQUD7qMAAAAAAAAA\nAAAwwRQAAQABAAABTAAEwDZwHsEUABwAAQAAAUwAECABBQIIzAAAAAAAAAAA\nADDA5AABAAEAAAFMAATAK6wewOQAHAABAAABTAAQIAEFAznBAAAAAAAAAAAA\nMMDEAAEAAQAAAUwABMAwTx7AxAAcAAEAAAFMABAgAQUCcJQAAAAAAAAAAAAw\nwKQAAQABAAABTAAEwDSyHsCkABwAAQAAAUwAECABBQMNLQAAAAAAAAAAADDB\nBAABAAEAAAFMAATAKaIewQQAHAABAAABTAAQIAEFANk3AAAAAAAAAAAAMMBk\nAAEAAQAAAUwABMA3Ux7AZAAcAAEAAAFMABAgAQUBsfkAAAAAAAAAAAAwAAAp\nEAAAAAAAAAA=\n"
+    payload         = Base64.decode64(encoded_payload)
+
+    Net::DNS::Packet.parse(payload)
+  end
 end

metadata

@@ -1,16 +1,29 @@
 --- !ruby/object:Gem::Specification
 name: gitlab-net-dns
 version: !ruby/object:Gem::Version
-  version: 0.10.0
+  version: 0.12.0
 platform: ruby
 authors:
 - Marco Ceresa
 - Simone Carletti
-autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-12-16 00:00:00.000000000 Z
+date: 2025-05-03 00:00:00.000000000 Z
 dependencies:
+- !ruby/object:Gem::Dependency
+  name: logger
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: mocha
   requirement: !ruby/object:Gem::Requirement
@@ -129,7 +142,6 @@ homepage: https://gitlab.com/gitlab-org/gitlab-net-dns
 licenses:
 - Ruby
 metadata: {}
-post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -137,15 +149,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '2.1'
+      version: '3.0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.22
-signing_key:
+rubygems_version: 3.6.2
 specification_version: 4
 summary: Pure Ruby DNS library.
 test_files: