gitlab-labkit 1.3.2 → 1.3.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8d8908ac363f634bae5e4a138d34f9d9e83f44297d17ff370bacb0e573c88ceb
-  data.tar.gz: b9527b0b05d8b828a4b3bf890b35af60d9ae6e4e893e98558577d419f9bddb5a
+  metadata.gz: fd5d320e1c751f2fea87cb55e57c3353b1255c62a126e97eba24a60642c9e0e0
+  data.tar.gz: 543e2fcd663bf562cb23426f0903acbafcf7f2edb6738e12dcc420fdfd8e26aa
 SHA512:
-  metadata.gz: d658010fd0809876abf220bcdb5b401a19cde96a9ce25006ddf83efac22a6cc7b80bb633b0aa60becb3874138b88da5697898ba77f0b5a7edb199013392dfff5
-  data.tar.gz: c35a260846129ae94108189da9c844753a9937b7a8606ad8db043625cea8cf409a62483a1a80510c7b6fc3080de70afc87af888b9285cabef520c0d115f54fda
+  metadata.gz: af194fc50ac69e6136c8e849b3445ed8f99c53eb555745ef5c03cbc5f42fde12b0a8fa33bb402c1fc80895feb4e9fba4a5c49cf345c77be10cc977f92cb105c4
+  data.tar.gz: e0b1dd39d3002edd45eb722d0aebe86e800688d878ab5e3beb79d060f65d7a3637ef3d5fb6351b9d80d81156542dd25dc0e3c291c4b7ce0485a4114752e69644

data/lib/labkit/logging/field_validator/log_interceptor.rb

@@ -38,17 +38,20 @@ module Labkit
           callsite_path = normalize_path(location.path)
           return data unless callsite_path
 
-          all_fields = extract_string_keys(data)
           logger_class = self.class.name || 'AnonymousLogger'
 
-          all_fields.each do |field|
-            standard_field = Labkit::Fields::Deprecated.standard_field_for(field)
-            next unless standard_field
+          deprecated_lookup = Labkit::Fields::Deprecated.all
+          if data.is_a?(Hash)
+            data.each_key do |key|
+              key_str = key.to_s
+              standard_field = deprecated_lookup[key_str]
+              next unless standard_field
 
-            Registry.instance.record_offense(callsite_path, location.lineno, field, standard_field, logger_class)
+              Registry.instance.record_offense(callsite_path, location.lineno, key_str, standard_field, logger_class)
+            end
           end
 
-          Registry.instance.check_for_removed_offenses(callsite_path, all_fields, logger_class)
+          Registry.instance.check_for_removed_offenses(callsite_path, data, logger_class)
 
           data
         end
@@ -83,12 +86,6 @@ module Labkit
           start_idx += 1 if absolute_path[start_idx] == '/'
           absolute_path[start_idx..]
         end
-
-        def extract_string_keys(data)
-          return Set.new unless data.is_a?(Hash)
-
-          data.keys.to_set(&:to_s)
-        end
       end
     end
   end

data/lib/labkit/logging/field_validator/registry.rb

@@ -109,6 +109,12 @@ module Labkit
           config = Config.load
           config.fetch('offenses', [])
         end
+
+        def extract_string_keys(data)
+          return Set.new unless data.is_a?(Hash)
+
+          data.keys.to_set(&:to_s)
+        end
       end
     end
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: gitlab-labkit
 version: !ruby/object:Gem::Version
-  version: 1.3.2
+  version: 1.3.3
 platform: ruby
 authors:
 - Andrew Newdigate