packetgen 3.1.1 → 3.1.6

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

@@ -1,10 +1,10 @@
+# frozen_string_literal: true
+
 # This file is part of PacketGen
 # See https://github.com/sdaubert/packetgen for more informations
 # Copyright (C) 2016 Sylvain Daubert <sylvain.daubert@laposte.net>
 # This program is published under MIT license.
 
-# frozen_string_literal: true
-
 module PacketGen
   module Header
     class OSPFv2

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

@@ -1,10 +1,10 @@
+# frozen_string_literal: true
+
 # This file is part of PacketGen
 # See https://github.com/sdaubert/packetgen for more informations
 # Copyright (C) 2016 Sylvain Daubert <sylvain.daubert@laposte.net>
 # This program is published under MIT license.
 
-# frozen_string_literal: true
-
 module PacketGen
   module Header
     class OSPFv2

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

@@ -1,10 +1,10 @@
+# frozen_string_literal: true
+
 # This file is part of PacketGen
 # See https://github.com/sdaubert/packetgen for more informations
 # Copyright (C) 2016 Sylvain Daubert <sylvain.daubert@laposte.net>
 # This program is published under MIT license.
 
-# frozen_string_literal: true
-
 module PacketGen
   module Header
     class OSPFv2

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

@@ -1,10 +1,10 @@
+# frozen_string_literal: true
+
 # This file is part of PacketGen
 # See https://github.com/sdaubert/packetgen for more informations
 # Copyright (C) 2016 Sylvain Daubert <sylvain.daubert@laposte.net>
 # This program is published under MIT license.
 
-# frozen_string_literal: true
-
 module PacketGen
   module Header
     class OSPFv2
@@ -14,6 +14,8 @@ module PacketGen
       # * and {#advertising_router}.
       # @author Sylvain Daubert
       class LSR < Types::Fields
+        include Types::Fieldable
+
         # @!attribute type
         #  The type of the LSA to request.
         #  @return [Integer]

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

@@ -1,10 +1,10 @@
+# frozen_string_literal: true
+
 # This file is part of PacketGen
 # See https://github.com/sdaubert/packetgen for more informations
 # Copyright (C) 2016 Sylvain Daubert <sylvain.daubert@laposte.net>
 # This program is published under MIT license.
 
-# frozen_string_literal: true
-
 module PacketGen
   module Header
     class OSPFv2

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

@@ -1,10 +1,10 @@
+# frozen_string_literal: true
+
 # This file is part of PacketGen
 # See https://github.com/sdaubert/packetgen for more informations
 # Copyright (C) 2016 Sylvain Daubert <sylvain.daubert@laposte.net>
 # This program is published under MIT license.
 
-# frozen_string_literal: true
-
 module PacketGen
   module Header
     class OSPFv2
@@ -23,6 +23,8 @@ module PacketGen
       # LSA router payload}.
       # @author Sylvain Daubert
       class TosMetric < Types::Fields
+        include Types::Fieldable
+
         # @!attribute tos
         #  8-bit IP Type of Service that this metric refers to.
         #  @return [Integer]
@@ -52,6 +54,8 @@ module PacketGen
       # This class handles links in a {LSARouter LSA router payload}.
       # @author Sylvain Daubert
       class Link < Types::Fields
+        include Types::Fieldable
+
         # @!attribute id
         #  @return [IP::Addr]
         define_field :id, IP::Addr
@@ -136,6 +140,8 @@ module PacketGen
       # This class handles external links in {LSAASExternal LSA AS-External payloads}.
       # @author Sylvain Daubert
       class External < Types::Fields
+        include Types::Fieldable
+
         # @!attribute u8
         #  @return [Integer]
         define_field :u8, Types::Int8
@@ -204,9 +210,7 @@ module PacketGen
         private
 
         def record_from_hash(hsh)
-          unless hsh.key? :type
-            raise ArgumentError, 'hash should have :type key'
-          end
+          raise ArgumentError, 'hash should have :type key' unless hsh.key? :type
 
           klass = if @only_headers
                     LSAHeader

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

@@ -1,10 +1,10 @@
+# frozen_string_literal: true
+
 # This file is part of PacketGen
 # See https://github.com/sdaubert/packetgen for more informations
 # Copyright (C) 2016 Sylvain Daubert <sylvain.daubert@laposte.net>
 # This program is published under MIT license.
 
-# frozen_string_literal: true
-
 module PacketGen
   module Header
     class OSPFv2
@@ -29,13 +29,15 @@ module PacketGen
       # is also a base class for different LSA class, as {LSARouter}.
       # @author Sylvain Daubert
       class LSAHeader < Types::Fields
+        include Types::Fieldable
+
         # LSA Types
         TYPES = {
-          'Router'       => 1,
-          'Network'      => 2,
-          'Summary-IP'   => 3,
+          'Router' => 1,
+          'Network' => 2,
+          'Summary-IP' => 3,
           'Summary-ABSR' => 4,
-          'AS-External'  => 5
+          'AS-External' => 5
         }.freeze
 
         # @!attribute age
@@ -75,7 +77,6 @@ module PacketGen
         # Compute and set Fletcher-16 checksum on LSA
         # @return [Integer]
         def calc_checksum
-          self.checksum = 0
           bytes = to_s[2..-1].unpack('C*')
 
           c0 = c1 = 0

data/lib/packetgen/header/ospfv3.rb

@@ -1,10 +1,10 @@
+# frozen_string_literal: true
+
 # This file is part of PacketGen
 # See https://github.com/sdaubert/packetgen for more informations
 # Copyright (C) 2016 Sylvain Daubert <sylvain.daubert@laposte.net>
 # This program is published under MIT license.
 
-# frozen_string_literal: true
-
 module PacketGen
   module Header
     # This class supports OSPFv3 (RFC 5340).

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

@@ -1,10 +1,10 @@
+# frozen_string_literal: true
+
 # This file is part of PacketGen
 # See https://github.com/sdaubert/packetgen for more informations
 # Copyright (C) 2016 Sylvain Daubert <sylvain.daubert@laposte.net>
 # This program is published under MIT license.
 
-# frozen_string_literal: true
-
 module PacketGen
   module Header
     class OSPFv3

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

@@ -1,10 +1,10 @@
+# frozen_string_literal: true
+
 # This file is part of PacketGen
 # See https://github.com/sdaubert/packetgen for more informations
 # Copyright (C) 2016 Sylvain Daubert <sylvain.daubert@laposte.net>
 # This program is published under MIT license.
 
-# frozen_string_literal: true
-
 module PacketGen
   module Header
     class OSPFv3

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

@@ -1,10 +1,10 @@
+# frozen_string_literal: true
+
 # This file is part of PacketGen
 # See https://github.com/sdaubert/packetgen for more informations
 # Copyright (C) 2016 Sylvain Daubert <sylvain.daubert@laposte.net>
 # This program is published under MIT license.
 
-# frozen_string_literal: true
-
 module PacketGen
   module Header
     class OSPFv3
@@ -17,6 +17,8 @@ module PacketGen
       #   array consumes ((PrefixLength + 31) / 32) 32-bit words.
       # @author Sylvain Daubert
       class IPv6Prefix < Types::Fields
+        include Types::Fieldable
+
         # @!attribute length
         #  Prefix length, in bits
         #  @return [Integer]

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

@@ -1,10 +1,10 @@
+# frozen_string_literal: true
+
 # This file is part of PacketGen
 # See https://github.com/sdaubert/packetgen for more informations
 # Copyright (C) 2016 Sylvain Daubert <sylvain.daubert@laposte.net>
 # This program is published under MIT license.
 
-# frozen_string_literal: true
-
 module PacketGen
   module Header
     class OSPFv3

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

@@ -1,10 +1,10 @@
+# frozen_string_literal: true
+
 # This file is part of PacketGen
 # See https://github.com/sdaubert/packetgen for more informations
 # Copyright (C) 2016 Sylvain Daubert <sylvain.daubert@laposte.net>
 # This program is published under MIT license.
 
-# frozen_string_literal: true
-
 module PacketGen
   module Header
     class OSPFv3
@@ -27,6 +27,8 @@ module PacketGen
       # * and a 32-bit {#advertising_router} field.
       # @author Sylvain Daubert
       class LSR < Types::Fields
+        include Types::Fieldable
+
         # @!attribute reserved
         #  reserved field.
         #  @return [Integer]

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

@@ -1,10 +1,10 @@
+# frozen_string_literal: true
+
 # This file is part of PacketGen
 # See https://github.com/sdaubert/packetgen for more informations
 # Copyright (C) 2016 Sylvain Daubert <sylvain.daubert@laposte.net>
 # This program is published under MIT license.
 
-# frozen_string_literal: true
-
 module PacketGen
   module Header
     class OSPFv3

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

@@ -1,16 +1,18 @@
+# frozen_string_literal: true
+
 # This file is part of PacketGen
 # See https://github.com/sdaubert/packetgen for more informations
 # Copyright (C) 2016 Sylvain Daubert <sylvain.daubert@laposte.net>
 # This program is published under MIT license.
 
-# frozen_string_literal: true
-
 module PacketGen
   module Header
     class OSPFv3
       # This class handles links in a {LSARouter OSPFv3 LSA router payload}.
       # @author Sylvain Daubert
       class Link < Types::Fields
+        include Types::Fieldable
+
         # @!attribute type
         #  @return [Integer]
         define_field :type, Types::Int8
@@ -197,9 +199,7 @@ module PacketGen
         private
 
         def record_from_hash(hsh)
-          unless hsh.key? :type
-            raise ArgumentError, 'hash should have :type key'
-          end
+          raise ArgumentError, 'hash should have :type key' unless hsh.key? :type
 
           klass = if @only_headers
                     LSAHeader

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

@@ -1,10 +1,10 @@
+# frozen_string_literal: true
+
 # This file is part of PacketGen
 # See https://github.com/sdaubert/packetgen for more informations
 # Copyright (C) 2016 Sylvain Daubert <sylvain.daubert@laposte.net>
 # This program is published under MIT license.
 
-# frozen_string_literal: true
-
 module PacketGen
   module Header
     class OSPFv3
@@ -29,15 +29,17 @@ module PacketGen
       # But this class is also a base class for different LSA class, as {LSARouter}.
       # @author Sylvain Daubert
       class LSAHeader < Types::Fields
+        include Types::Fieldable
+
         # LSA known types
         TYPES = {
-          'Router'            => 0x2001,
-          'Network'           => 0x2002,
+          'Router' => 0x2001,
+          'Network' => 0x2002,
           'Inter-Area-Prefix' => 0x2003,
           'Inter-Area-Router' => 0x2004,
-          'AS-External'       => 0x4005,
-          'NSSA'              => 0x2007,
-          'Link'              => 0x0008,
+          'AS-External' => 0x4005,
+          'NSSA' => 0x2007,
+          'Link' => 0x0008,
           'Intra-Area-Prefix' => 0x2009
         }.freeze
 
@@ -76,7 +78,6 @@ module PacketGen
         # Compute and set Fletcher-16 checksum on LSA
         # @return [Integer]
         def calc_checksum
-          self.checksum = 0
           bytes = to_s[2..-1].unpack('C*')
 
           c0 = c1 = 0

data/lib/packetgen/header/snmp.rb

@@ -1,11 +1,11 @@
 # coding: utf-8
+# frozen_string_literal: true
+
 # This file is part of PacketGen
 # See https://github.com/sdaubert/packetgen for more informations
 # Copyright (C) 2016 Sylvain Daubert <sylvain.daubert@laposte.net>
 # This program is published under MIT license.
 
-# frozen_string_literal: true
-
 module PacketGen
   module Header
     # Dissect error
@@ -22,6 +22,8 @@ module PacketGen
       # Configuration sinks listen to this port
       UDP_PORT2 = 162
 
+      # rubocop:disable Naming/ConstantName
+
       PDU_GET      = 0
       PDU_NEXT     = 1
       PDU_RESPONSE = 2
@@ -31,27 +33,28 @@ module PacketGen
       PDU_INFORM   = 6
       PDU_TRAPv2   = 7
       PDU_REPORT   = 8
+      # rubocop:enable Naming/ConstantName
 
       ERRORS = {
-        'no_error'              => 0,
-        'too_big'               => 1,
-        'no_such_name'          => 2,
-        'bad_value'             => 3,
-        'read_only'             => 4,
-        'generic_error'         => 5,
-        'no_access'             => 6,
-        'wrong_type'            => 7,
-        'wrong_length'          => 8,
-        'wrong_encoding'        => 9,
-        'wrong_value'           => 10,
-        'no_creation'           => 11,
-        'inconsistent_value'    => 12,
+        'no_error' => 0,
+        'too_big' => 1,
+        'no_such_name' => 2,
+        'bad_value' => 3,
+        'read_only' => 4,
+        'generic_error' => 5,
+        'no_access' => 6,
+        'wrong_type' => 7,
+        'wrong_length' => 8,
+        'wrong_encoding' => 9,
+        'wrong_value' => 10,
+        'no_creation' => 11,
+        'inconsistent_value' => 12,
         'ressource_unavailable' => 13,
-        'commit_failed'         => 14,
-        'undo_failed'           => 15,
-        'authorization_error'   => 16,
-        'not_writable'          => 17,
-        'inconsistent_name'     => 18
+        'commit_failed' => 14,
+        'undo_failed' => 15,
+        'authorization_error' => 16,
+        'not_writable' => 17,
+        'inconsistent_name' => 18
       }.freeze
 
       # Class to handle SNMP VarBind
@@ -162,13 +165,13 @@ module PacketGen
                  implicit: SNMP::PDU_TRAPv1, constructed: true,
                  content: [objectid(:enterprise),
                            octet_string(:agent_addr),
-                           integer(:generic_trap, enum: { 'cold_start'        => 0,
-                                                          'warm_start'        => 1,
-                                                          'link_down'         => 2,
-                                                          'link_up'           => 3,
-                                                          'auth_failure'      => 4,
+                           integer(:generic_trap, enum: { 'cold_start' => 0,
+                                                          'warm_start' => 1,
+                                                          'link_down' => 2,
+                                                          'link_up' => 3,
+                                                          'auth_failure' => 4,
                                                           'egp_neighbor_loss' => 5,
-                                                          'specific'          => 6 }),
+                                                          'specific' => 6 }),
                            integer(:specific_trap),
                            integer(:timestamp),
                            model(:varbindlist, VariableBindings)]
@@ -243,7 +246,8 @@ module PacketGen
       end
 
       sequence :message,
-               content: [integer(:version, value: 'v2c',
+               content: [integer(:version,
+                                 value: 'v2c',
                                  enum: { 'v1' => 0, 'v2c' => 1, 'v2' => 2, 'v3' => 3 }),
                          octet_string(:community, value: 'public'),
                          model(:data, PDUs)]
@@ -291,8 +295,8 @@ module PacketGen
           str << data.chosen_value.inspect(1)
           begin
             str << Inspect.inspect_body(self[:message].to_der, 'ASN.1 DER')
-          rescue StandardError => ex
-            raise unless ex.message =~ /TAG.*not handled/
+          rescue StandardError => e
+            raise unless e.message.match?(/TAG.*not handled/)
           end
           str
         end