packetgen 3.2.1 → 3.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 79dc573da10eb15bbd9f38e0c84ecdf9a92d076980628635d1b9e240e3ca2210
-  data.tar.gz: 0ae2bafa08e5a4a0b176276a141b8c8326265873bb20d86a75e07bb7018275ac
+  metadata.gz: de85a6f18ac001823cdbd0802c8946ead811be724c9069266b78c493d99feb6b
+  data.tar.gz: ef73ca367e87bb47ca4f37ee387c7a73d0ad0f19cebaca151443560f42508559
 SHA512:
-  metadata.gz: 7c405a57b79c4a9bf0d9032b34d7381b33e8c0705c554023f6a8131c19c48fb0118201d94da62e4f74b9d53551cdbcd434cb6363f27a84e5c3e8459cb28d613b
-  data.tar.gz: 835969235415864e05b227dd5f0fb48535e3f414e60852cf788d6d90bf29be9a3a8dcf9b636ce8d500d29720753246e11459da925fea3f72f6cb8e11291565b8
+  metadata.gz: '01605812af7b7d53111bfdc0d4d095cb6f0df4975634db17700d7c15c6acc63ba82585e6c5a5d2d93c130d566f45bb62d9ff3709c27e0f7cf4ea0c20ba7c172f'
+  data.tar.gz: e4a54c74e032ee12d744f94c657416591c0f056a005da4b104c017294d73819596a28ec59c71597f8d0f2398da79a8b7784809d5e30cd5da9d53fc655101e2f4

data/lib/packetgen/capture.rb

@@ -27,6 +27,11 @@ module PacketGen
     # @return [Array<String>]
     attr_reader :raw_packets
 
+    # Get timestamps associated with {#packets} and {#raw_packets}
+    # @return [Array<Time>]
+    # @since 3.3.0
+    attr_reader :timestamps
+
     # Get interface name
     # @return [String]
     attr_reader :iface
@@ -54,6 +59,7 @@ module PacketGen
 
       @packets     = []
       @raw_packets = []
+      @timestamps = []
       set_options iface, max, timeout, filter, promisc, parse, snaplen, monitor
     end
 
@@ -61,16 +67,18 @@ module PacketGen
     # @see {#initialize} for parameters
     # @yieldparam [Packet,String] packet if a block is given, yield each
     #    captured packet (Packet or raw data String, depending on +:parse+ option)
+    # @yieldparam [Time] timestamp packet timestamp
     # @since 3.0.0 arguments are kwargs and no more a hash
     # @since 3.1.5 add monitor argument
+    # @since 3.3.0 add packet timestamp as second yield parameter
     # @author Sylvain Daubert
     # @author optix2000 - add monitor argument
     def start(iface: nil, max: nil, timeout: nil, filter: nil, promisc: false, parse: true, snaplen: nil, monitor: nil, &block)
       set_options iface, max, timeout, filter, promisc, parse, snaplen, monitor
 
       @cap_thread = Thread.new do
-        PCAPRUBWrapper.capture(**capture_args) do |packet_data|
-          add_packet(packet_data, &block)
+        PCAPRUBWrapper.capture(**capture_args) do |packet|
+          add_packet(packet, &block)
           break if defined?(@max) && (raw_packets.size >= @max)
         end
       end
@@ -119,18 +127,21 @@ module PacketGen
       PCAPRUBWrapper.filter_on(pcap: pcap, filter: filter)
     end
 
-    def add_packet(data, &block)
-      raw_packets << data
+    def add_packet(packet, &block)
+      raw_packets << packet.data
+      ts = Time.at(packet.time, packet.microsec.to_r, :usec)
+      timestamps << ts
+
       if @parse
         begin
-          packet = Packet.parse(data)
+          packet = Packet.parse(packet.data)
         rescue ParseError
-          packet = UnknownPacket.new.parse(data)
+          packet = UnknownPacket.new.parse(packet.data)
         end
         packets << packet
-        block&.call(packet)
+        block&.call(packet, ts)
       elsif block
-        yield data
+        yield data, ts
       end
     end
   end

data/lib/packetgen/header/dhcpv6/duid.rb

@@ -58,6 +58,12 @@ module PacketGen
         def to_human
           "DUID<#{type},#{body.inspect}>"
         end
+
+        # Get human-readable type
+        # @return [String]
+        def human_type
+          self[:type].to_human
+        end
       end
 
       # rubocop:disable Naming/ClassAndModuleCamelCase

data/lib/packetgen/header/dns/name.rb

@@ -67,9 +67,9 @@ module PacketGen
               @pointer = str[start, 2]
               break
             else
-              label = add_label_from(str[start..-1])
+              label = add_label_from(str[start..])
               start += label.sz
-              break if label.length.zero? || str[start..-1].length.zero?
+              break if label.empty? || str[start..].empty?
             end
           end
           # force resolution of compressed names
@@ -110,7 +110,7 @@ module PacketGen
           ptr = index & mask
           name = Name.new
           name.dns = @dns
-          @pointer_name = name.read(self.dns.to_s[ptr..-1]).to_human
+          @pointer_name = name.read(self.dns.to_s[ptr..]).to_human
         end
 
         def record_from_hash(_hsh)

data/lib/packetgen/header/dns/rr.rb

@@ -108,7 +108,7 @@ module PacketGen
           name.dns = self[:name].dns
 
           pref = Types::Int16.new.read(self[:rdata][0, 2])
-          exchange = name.read(self[:rdata][2..-1]).to_human
+          exchange = name.read(self[:rdata][2..]).to_human
 
           '%u %s' % [pref.to_i, exchange]
         end
@@ -118,7 +118,7 @@ module PacketGen
           name = Name.new
           name.dns = self[:name].dns
           mname = name.read(self[:rdata]).dup
-          rname = name.read(self[:rdata][mname.sz..-1])
+          rname = name.read(self[:rdata][mname.sz..])
 
           serial = Types::Int32.new.read(self[:rdata][mname.sz + rname.sz, 4])
           refresh = Types::Int32.new.read(self[:rdata][mname.sz + rname.sz + 4, 4])

data/lib/packetgen/header/dot11.rb

@@ -403,7 +403,7 @@ module PacketGen
         define_applicable_fields
         if fcs
           old_read str[0...-4]
-          self[:fcs].read str[-4..-1]
+          self[:fcs].read str[-4..]
         else
           old_read str
         end

data/lib/packetgen/header/http/headers.rb

@@ -34,13 +34,28 @@ module PacketGen
 
               k, v = h.split(':', 2)
               [k, v.strip]
-            end.reject(&:nil?).to_h
+            end.compact.to_h
           when Hash
             @data = s_or_h
           end
           self
         end
 
+        # Get header value from its name
+        # @param [String] header header name
+        # @return [String] header value
+        def [](header)
+          data[header]
+        end
+
+        # Say if +self+ include +header+ header
+        # @param [String] header header name
+        # @return [Boolean]
+        def header?(header)
+          data.key?(header)
+        end
+        alias has_header? header?
+
         # Get binary string.
         # @return [String]
         def to_s

data/lib/packetgen/header/http/request.rb

@@ -107,7 +107,7 @@ module PacketGen
 
         def headers_and_payload_from_lines(lines)
           if (data_index = lines.find_index(''))
-            data    = lines[data_index + 1..-1].join("\n")
+            data    = lines[data_index + 1..].join("\n")
             headers = lines[0..data_index - 1].join("\n")
           else
             headers = lines.join("\n")

data/lib/packetgen/header/http/response.rb

@@ -126,7 +126,7 @@ module PacketGen
 
           self[:version].read first_line[0]
           self[:status_code].read first_line[1]
-          self[:status_mesg].read first_line[2..-1].join(' ')
+          self[:status_mesg].read first_line[2..].join(' ')
         end
 
         def raise_on_bad_version_status

data/lib/packetgen/header/ip/option.rb

@@ -18,7 +18,7 @@ module PacketGen
         # @return [self]
         #   array << '192.168.1.12'
         def push(addr)
-          addr = addr.is_a?(Addr) ? addr : Addr.new.from_human(addr)
+          addr = Addr.new.from_human(addr) unless addr.is_a?(Addr)
           super(addr)
         end
       end
@@ -77,7 +77,7 @@ module PacketGen
           Option.constants.each do |cst|
             next unless cst.to_s.end_with? '_TYPE'
 
-            optname = cst.to_s.sub(/_TYPE/, '')
+            optname = cst.to_s.sub('_TYPE', '')
             @types[optname] = Option.const_get(cst)
           end
           @types
@@ -86,25 +86,23 @@ module PacketGen
         # Factory to build an option from its type
         # @return [Option]
         def self.build(options={})
-          type = options.delete(:type)
+          type = options[:type]
           klass = case type
                   when String
                     types.key?(type) ? IP.const_get(type) : self
-                  when Integer
-                    types.value?(type) ? IP.const_get(types.key(type)) : self
                   else
-                    self
+                    types.value?(type) ? IP.const_get(types.key(type.to_i)) : self
                   end
+          options.delete(:type) if klass != self
           klass.new(options)
         end
 
         def initialize(options={})
-          unless options[:type]
-            opt_name = self.class.to_s.gsub(/.*::/, '')
-            options[:type] = Option.const_get("#{opt_name}_TYPE") if Option.const_defined? "#{opt_name}_TYPE"
-          end
+          options[:type] = class2type unless options[:type]
+
           super
-          self.length = sz if respond_to?(:length) && options[:length].nil?
+          initialize_length_if_needed(options)
+          initialize_data_if_needed(options)
         end
 
         # Get binary string. Set {#length} field.
@@ -121,6 +119,25 @@ module PacketGen
           str << ":#{self[:data].to_s.inspect}" if respond_to?(:length) && (length > 2) && !self[:data].to_s.empty?
           str
         end
+
+        private
+
+        def class2type
+          opt_name = self.class.to_s.gsub(/.*::/, '')
+          Option.const_get("#{opt_name}_TYPE") if Option.const_defined? "#{opt_name}_TYPE"
+        end
+
+        def initialize_length_if_needed(options)
+          self.length = sz if respond_to?(:length) && options[:length].nil?
+        end
+
+        def initialize_data_if_needed(options)
+          return unless fields.include?(:data) && self[:data].respond_to?(:from_human) && options.key?(:data)
+
+          # Force data if data is set in options but not length
+          self.length += options[:data].size
+          self[:data].from_human(options[:data])
+        end
       end
 
       # End-of-option-List IP option
@@ -173,6 +190,10 @@ module PacketGen
         #  16-bit stream ID
         #  @return [Integer]
         define_field :id, Types::Int16
+
+        def to_human
+          super << ":#{self.id}"
+        end
       end
 
       # Router Alert IP option
@@ -183,6 +204,10 @@ module PacketGen
         #  16-bit value. Should be 0.
         #  @return [Integer]
         define_field :value, Types::Int16, default: 0
+
+        def to_human
+          super << ":#{self.value}"
+        end
       end
     end
   end

data/lib/packetgen/header/ipv6/addr.rb

@@ -107,7 +107,7 @@ module PacketGen
         # @return [self]
         #   array << '2001:1234::125'
         def push(addr)
-          addr = addr.is_a?(Addr) ? addr : Addr.new.from_human(addr)
+          addr = Addr.new.from_human(addr) unless addr.is_a?(Addr)
           super(addr)
         end
       end

data/lib/packetgen/header/ospfv2/lsa_header.rb

@@ -78,7 +78,7 @@ module PacketGen
         # @return [Integer]
         def calc_checksum
           c0 = c1 = 0
-          to_s[2..-1].unpack('C*').each do |byte|
+          to_s[2..].unpack('C*').each do |byte|
             c0 += byte
             c1 += c0
           end

data/lib/packetgen/header/ospfv3/lsa_header.rb

@@ -79,7 +79,7 @@ module PacketGen
         # @return [Integer]
         def calc_checksum
           c0 = c1 = 0
-          to_s[2..-1].unpack('C*').each do |byte|
+          to_s[2..].unpack('C*').each do |byte|
             c0 += byte
             c1 += c0
           end

data/lib/packetgen/header/snmp.rb

@@ -74,6 +74,18 @@ module PacketGen
       # @author Sylvain Daubert
       class VariableBindings < RASN1::Model
         sequence_of :bindings, VarBind
+
+        # Get 'idx'th element from the list
+        # @return [VarBind,nil]
+        def [](idx)
+          value[idx]
+        end
+
+        # Get element counts in list
+        # @return [Integer]
+        def size
+          value.size
+        end
       end
 
       # Class to handle GetRequest PDU
@@ -119,6 +131,10 @@ module PacketGen
                            integer(:error, value: 0, enum: ERRORS),
                            integer(:error_index, value: 0),
                            model(:varbindlist, VariableBindings)]
+
+        def initialize(args={})
+          super
+        end
       end
 
       # Class to handle GetNextRequest PDU
@@ -264,11 +280,12 @@ module PacketGen
         data.chosen = options[:chosen_pdu] if options[:chosen_pdu]
         return unless options[:pdu]
 
-        data.root.value[data.chosen] = data.root.chosen_value.class.new(options[:pdu])
+        klass = data.root.chosen_value.class
+        data.root.value[data.chosen] = klass.new(options[:pdu])
       end
 
       # accessor to data payload
-      # @return [GetRequest]
+      # @return [ASN1::Types::Choice]
       def data
         @elements[:data]
       end

data/lib/packetgen/header/tcp/options.rb

@@ -24,7 +24,7 @@ module PacketGen
           Option.constants.each do |cst|
             next unless cst.to_s.end_with? '_KIND'
 
-            optname = cst.to_s.sub(/_KIND/, '')
+            optname = cst.to_s.sub('_KIND', '')
             @klasses[Option.const_get(cst)] = TCP.const_get(optname)
           end
           @klasses

data/lib/packetgen/headerable.rb

@@ -46,7 +46,7 @@ module PacketGen
     def method_name
       return @method_name if defined? @method_name
 
-      @method_name = protocol_name.downcase.gsub(/::/, '_')
+      @method_name = protocol_name.downcase.gsub('::', '_')
     end
 
     # @abstract Should be redefined by subclasses. This method should check invariant

data/lib/packetgen/inspect.rb

@@ -36,7 +36,8 @@ module PacketGen
     # @param [Integer] hexsize
     # @return [String]
     def self.int_dec_hex(value, hexsize)
-      "%-16s (0x%0#{hexsize}x)" % [value.to_i, value.to_i]
+      fmt = "%-16s (0x%0#{hexsize}x)"
+      fmt % [value.to_i, value.to_i]
     end
 
     # @param [String] str
@@ -44,7 +45,8 @@ module PacketGen
     # @param [Integer] hexsize
     # @return [String]
     def self.enum_human_hex(str, int, hexsize)
-      "%-16s (0x%0#{hexsize}x)" % [str, int]
+      fmt = "%-16s (0x%0#{hexsize}x)"
+      fmt % [str, int]
     end
 
     # Simple formatter to inspect an attribute
@@ -90,7 +92,8 @@ module PacketGen
       val = case attr
             when RASN1::Types::Enumerated
               hexsize = attr.value_size * 2
-              "%-16s (0x%0#{hexsize}x)" % [attr.value, attr.to_i]
+              fmt = "%-16s (0x%0#{hexsize}x)"
+              fmt % [attr.value, attr.to_i]
             when RASN1::Types::Integer
               int_dec_hex(attr.value, attr.value_size * 2)
             when RASN1::Model

data/lib/packetgen/packet.rb

@@ -81,7 +81,9 @@ module PacketGen
     # @see Capture#initialize
     # @yieldparam [Packet,String] packet if a block is given, yield each
     #    captured packet (Packet or raw data String, depending on +:parse+ option)
+    # @yieldparam [Time] timestamp packet timestamp
     # @return [Array<Packet>] captured packet
+    # @since 3.3.0 add packet timestamp as second yield parameter
     def self.capture(**kwargs, &block)
       capture = Capture.new(**kwargs)
       if block

data/lib/packetgen/pcapng/file.rb

@@ -21,11 +21,11 @@ module PacketGen
       }.freeze
 
       # @private
-      BLOCK_TYPES = PcapNG.constants(false).select { |c| c.to_s.include?('_TYPE') }.map do |c|
+      BLOCK_TYPES = PcapNG.constants(false).select { |c| c.to_s.include?('_TYPE') }.to_h do |c|
         type_value = PcapNG.const_get(c).to_i
         klass = PcapNG.const_get(c.to_s.delete_suffix('_TYPE'))
         [type_value, klass]
-      end.to_h.freeze
+      end.freeze
 
       # Get file sections
       # @return [Array]

data/lib/packetgen/pcaprub_wrapper.rb

@@ -44,7 +44,7 @@ module PacketGen
     # @param [Boolean] promisc
     # @param [String] filter BPF filter
     # @param [Boolean] monitor
-    # @yieldparam [String] packet_data binary packet data
+    # @yieldparam [String] packet_data packet data
     # @return [void]
     # @author Sylvain Daubert
     # @author optix2000 - add support for setting monitor mode
@@ -52,7 +52,7 @@ module PacketGen
     def self.capture(iface:, snaplen: DEFAULT_SNAPLEN, promisc: DEFAULT_PROMISC, filter: nil, monitor: nil, &block)
       pcap = self.open_iface(iface: iface, snaplen: snaplen, promisc: promisc, monitor: monitor)
       pcap.setfilter filter unless filter.nil?
-      pcap.each(&block)
+      pcap.each_packet(&block)
     end
 
     # Inject given data onto wire

data/lib/packetgen/proto.rb

@@ -19,7 +19,7 @@ module PacketGen
       proto_constants = Socket.constants.grep(/IPPROTO_/)
       @cache = {}
       proto_constants.each do |const_sym|
-        name = const_sym.to_s[8..-1].downcase
+        name = const_sym.to_s[8..].downcase
         number = Socket.const_get(const_sym)
         @cache[name] = number
       end

data/lib/packetgen/types/array.rb

@@ -154,26 +154,19 @@ module PacketGen
         self
       end
 
-      # rubocop:disable Metrics/CyclomaticComplexity
-
-      # Populate object from a string
-      # @param [String] str
+      # Populate object from a string or from an array of hashes
+      # @param [String, Array<Hash>] data
       # @return [self]
-      def read(str)
+      def read(data)
         clear
-        return self if str.nil?
-        return self if @counter&.to_i&.zero?
-
-        str = read_with_length_from(str)
-        until str.empty?
-          obj = create_object_from_str(str)
-          @array << obj
-          str.slice!(0, obj.sz)
-          break if @counter && self.size == @counter.to_i
+        case data
+        when ::Array
+          read_from_array(data)
+        else
+          read_from_string(data)
         end
         self
       end
-      # rubocop:enable Metrics/CyclomaticComplexity
 
       # Get size in bytes
       # @return [Integer]
@@ -201,6 +194,28 @@ module PacketGen
 
       private
 
+      # rubocop:disable Metrics/CyclomaticComplexity
+
+      def read_from_string(str)
+        return self if str.nil? || @counter&.to_i&.zero?
+
+        str = read_with_length_from(str)
+        until str.empty? || (@counter && self.size == @counter.to_i)
+          obj = create_object_from_str(str)
+          @array << obj
+          str.slice!(0, obj.sz)
+        end
+      end
+      # rubocop:enable Metrics/CyclomaticComplexity
+
+      def read_from_array(ary)
+        return self if ary.empty?
+
+        ary.each do |hsh|
+          self << hsh
+        end
+      end
+
       def record_from_hash(hsh)
         obj_klass = self.class.set_of_klass
         raise NotImplementedError, 'class should define #record_from_hash or declare type of elements in set with .set_of' unless obj_klass

data/lib/packetgen/types/fields.rb

@@ -322,12 +322,14 @@ module PacketGen
             define << "def #{name}; self[:#{name}].to_i; end"
             define << "def #{name}=(val) self[:#{name}].value = val; end"
           else
+            # rubocop:disable Layout/LineContinuationLeadingSpace
             define << "def #{name}\n" \
                       "  to_and_from_human?(:#{name}) ? self[:#{name}].to_human : self[:#{name}]\n" \
                       'end'
             define << "def #{name}=(val)\n" \
                       "  to_and_from_human?(:#{name}) ? self[:#{name}].from_human(val) : self[:#{name}].read(val)\n" \
                       'end'
+            # rubocop:enable Layout/LineContinuationLeadingSpace
           end
 
           define.delete_at(1) if instance_methods.include? "#{name}=".to_sym
@@ -479,7 +481,7 @@ module PacketGen
         fields.each do |field|
           next unless present?(field)
 
-          obj = self[field].read str[start..-1]
+          obj = self[field].read str[start..]
           start += self[field].sz
           self[field] = obj unless obj == self[field]
         end
@@ -523,7 +525,7 @@ module PacketGen
       # Return object as a hash
       # @return [Hash] keys: attributes, values: attribute values
       def to_h
-        fields.map { |f| [f, @fields[f].to_human] }.to_h
+        fields.to_h { |f| [f, @fields[f].to_human] }
       end
 
       # Get offset of given field in {Fields} structure.

data/lib/packetgen/unknown_packet.rb

@@ -61,7 +61,8 @@ module PacketGen
 
     # @return [String]
     def inspect
-      # TODO
+      str = Inspect.dashed_line(self.class)
+      str << Inspect.inspect_body(body)
     end
 
     # equality if {#to_s} is equal
@@ -74,7 +75,7 @@ module PacketGen
     # @return [Boolean]
     def ===(other)
       case other
-      when UnknwonPacket
+      when UnknownPacket
         self == other
       else
         false

data/lib/packetgen/utils/arp_spoofer.rb

@@ -112,9 +112,11 @@ module PacketGen
 
       # Say if spoofing on given target is active or not
       # @param [String] target_ip target IP address
-      # @return [Boolean,nil]
+      # @return [Boolean]
       def active?(target_ip)
+        # rubocop:disable Style/ReturnNilInPredicateMethodDefinition
         return unless @targets.key?(target_ip)
+        # rubocop:enable Style/ReturnNilInPredicateMethodDefinition
 
         @targets[target_ip][:active]
       end

data/lib/packetgen/utils.rb

@@ -19,29 +19,38 @@ module PacketGen
     # @private
     ARP_FILTER = 'arp src %<ipaddr>s and ether dst %<hwaddr>s'
     # @private
-    MITM_FILTER = '((ip src %<target1>s and not ip dst %<local_ip>s) or' \
-                  ' (ip src %<target2>s and not ip dst %<local_ip>s) or' \
-                  ' (ip dst %<target1>s and not ip src %<local_ip>s) or' \
-                  ' (ip dst %<target2>s and not ip src %<local_ip>s))' \
-                  ' and ether dst %<local_mac>s'
+    MITM_FILTER = '((ip src %<target1>s and not ip dst %<local_ip>s) or ' \
+                  '(ip src %<target2>s and not ip dst %<local_ip>s) or ' \
+                  '(ip dst %<target1>s and not ip src %<local_ip>s) or ' \
+                  '(ip dst %<target2>s and not ip src %<local_ip>s)) ' \
+                  'and ether dst %<local_mac>s'
+
+    # @private
+    ARP_PATH = '/usr/sbin/arp'
+    # @private
+    IP_PATH = '/usr/bin/ip'
+    # @private
+    ARP_LINE_RE = /\((\d+\.\d+\.\d+\.\d+)\) at (([a-fA-F0-9]{2}:){5}[a-fA-F0-9]{2})(?: \[ether\])? on (\w+)/.freeze
+    # @private
+    IP_LINE_RE = /^(\d+\.\d+\.\d+\.\d+) dev (\w+) lladdr (([a-fA-F0-9]{2}:){5}[a-fA-F0-9]{2})/.freeze
 
     # Get local ARP cache
     # @return [Hash] key: IP address, value: array containing MAC address and
     #    interface name
     def self.arp_cache
-      return self.cache_from_arp_command if File.exist?('/usr/sbin/arp')
-      return self.cache_from_ip_command if File.exist?('/usr/bin/ip')
+      return self.cache_from_arp_command if File.exist?(ARP_PATH)
+      return self.cache_from_ip_command if File.exist?(IP_PATH)
 
       {}
     end
 
     # @private
-    def self.cache_from_arp_command
-      raw_cache = `/usr/sbin/arp -an`
+    def self.cache_from_arp_command(raw_cache=nil)
+      raw_cache ||= `#{ARP_PATH} -an`
 
       cache = {}
       raw_cache.split("\n").each do |line|
-        match = line.match(/\((\d+\.\d+\.\d+\.\d+)\) at (([a-fA-F0-9]{2}:){5}[a-fA-F0-9]{2})(?: \[ether\])? on (\w+)/)
+        match = line.match(ARP_LINE_RE)
         cache[match[1]] = [match[2], match[4]] if match
       end
 
@@ -49,12 +58,12 @@ module PacketGen
     end
 
     # @private
-    def self.cache_from_ip_command
-      raw_cache = `ip neigh`
+    def self.cache_from_ip_command(raw_cache=nil)
+      raw_cache ||= `#{IP_PATH} neigh`
 
       cache = {}
       raw_cache.split("\n").each do |line|
-        match = line.match(/^(\d+\.\d+\.\d+\.\d+) dev (\w+) lladdr (([a-fA-F0-9]{2}:){5}[a-fA-F0-9]{2})/)
+        match = line.match(IP_LINE_RE)
         cache[match[1]] = [match[3], match[2]] if match
       end
 
@@ -155,7 +164,7 @@ module PacketGen
     #   end
     # @since 2.2.0
     # @raise [RuntimeError] user don't have permission to capture packets on network device.
-    def self.mitm(target1, target2, options={})
+    def self.mitm(target1, target2, options={}, &block)
       options = { iface: PacketGen.default_iface }.merge(options)
 
       spoofer = Utils::ARPSpoofer.new(options)
@@ -168,7 +177,7 @@ module PacketGen
                             filter: MITM_FILTER % { target1: target1, target2: target2, local_ip: cfg.ipaddr(options[:iface]), local_mac: my_mac })
 
       spoofer.start_all
-      mitm_core(capture, target1, target2, my_mac, &proc)
+      mitm_core(capture, target1, target2, my_mac, &block)
       spoofer.stop_all
     end
 

data/lib/packetgen/version.rb

@@ -10,5 +10,5 @@
 # @author Sylvain Daubert
 module PacketGen
   # PacketGen version
-  VERSION = '3.2.1'
+  VERSION = '3.3.0'
 end

data/lib/packetgen.rb

@@ -64,8 +64,10 @@ module PacketGen
   # Shortcut for {Packet.capture}
   # Same arguments as {Capture#initialize}
   # @see Capture#initialize
-  # @yieldparam [Packet] packet
+  # @yieldparam [Packet,String] packet
+  # @yieldparam [Time] timestamp
   # @return [Array<Packet>]
+  # @since 3.3.0 add packet timestamp as second yield parameter
   def self.capture(**kwargs)
     Packet.capture(**kwargs) { |packet| yield packet if block_given? }
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: packetgen
 version: !ruby/object:Gem::Version
-  version: 3.2.1
+  version: 3.3.0
 platform: ruby
 authors:
 - Sylvain Daubert
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-12-27 00:00:00.000000000 Z
+date: 2023-08-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: interfacez
@@ -30,76 +30,28 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.12.0
+        version: 0.13.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.12.0
+        version: 0.13.0
 - !ruby/object:Gem::Dependency
   name: rasn1
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.8'
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 0.8.0
+        version: '0.12'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.8'
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 0.8.0
-- !ruby/object:Gem::Dependency
-  name: rake
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '13.0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '13.0'
-- !ruby/object:Gem::Dependency
-  name: rspec
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '3.7'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '3.7'
-- !ruby/object:Gem::Dependency
-  name: yard
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '0.9'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '0.9'
+        version: '0.12'
 description: |
   PacketGen is a network packet manipulation library. It allows reading, parsing
   and sending network packets with fun.
@@ -260,7 +212,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.5.0
+      version: 2.6.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="