appmap 0.102.0 → 0.102.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1df97ff2f7c62ce830193389327e102a89798678350dbc653aecc98157e502cb
-  data.tar.gz: ff14a72537014f549a5094b592a6dd28e9a6658679e0c411529d7c4c12491822
+  metadata.gz: 7535028ffd94cccfeb167a51005d03d3eb49bdab5e987d08eaaa41040f2ead72
+  data.tar.gz: 1e447679a5a3b0b690d9db333e4150e28c84c5bc0556f221a959331c16ac259f
 SHA512:
-  metadata.gz: 21781f0d7b9d61d72cd6b442b1105a1eabd19ba07ccca62bb8fd5acaede9e6db2a3cf5594a80c1270ff574d977b709c22031a12a88361cd36fe3013e5238cb03
-  data.tar.gz: bcc9fd8526da90172d0f7f159a1c4a61a0245e70d1d800f38b98e1d13f0a35ee26898c064107a0de5eb3f7d11d1eecf98c12df1f14a4c37cb72582a1a6bdcf1d
+  metadata.gz: 9356f03158736805234225c0a0368317658b4d2ad821ca67cb9c2582f5d7ada84a0c36edbbc71a26dab37509bcbc79c3e8bdaa16b64a76468702ccea068d4068
+  data.tar.gz: 592abd7344c26dbd07a6137aa99bf5b8ba21020569b796b6dcecb06a27a8bc24bb5079f9992d85cb6ebb9c43ab2d05690361dc1bc0e5c9dea52d78594290ebc6

data/CHANGELOG.md

@@ -1,3 +1,17 @@
+## [0.102.2](https://github.com/getappmap/appmap-ruby/compare/v0.102.1...v0.102.2) (2023-11-07)
+
+
+### Bug Fixes
+
+* Don't emit invalid HTTP status ([d30fb06](https://github.com/getappmap/appmap-ruby/commit/d30fb06223c6e30e0681c4fac2a9f8824d3c035f)), closes [/github.com/rails/rails/blob/febd21da3438d190f0c6ce296a114baee983d1e1/actioncable/lib/action_cable/connection/client_socket.rb#L71C23-L71C23](https://github.com//github.com/rails/rails/blob/febd21da3438d190f0c6ce296a114baee983d1e1/actioncable/lib/action_cable/connection/client_socket.rb/issues/L71C23-L71C23) [/github.com/faye/websocket-driver-ruby/blob/58d4ed742749dfb3b4f568df36a5365088361220/lib/websocket/driver/draft76.rb#L61](https://github.com//github.com/faye/websocket-driver-ruby/blob/58d4ed742749dfb3b4f568df36a5365088361220/lib/websocket/driver/draft76.rb/issues/L61)
+
+## [0.102.1](https://github.com/getappmap/appmap-ruby/compare/v0.102.0...v0.102.1) (2023-07-25)
+
+
+### Bug Fixes
+
+* Guard access to the :size method ([e0b53b2](https://github.com/getappmap/appmap-ruby/commit/e0b53b23d46f8467884c5c387a9081d98cb7cca7)), closes [#341](https://github.com/getappmap/appmap-ruby/issues/341)
+
 # [0.102.0](https://github.com/getappmap/appmap-ruby/compare/v0.101.0...v0.102.0) (2023-07-18)
 
 

data/appmap.gemspec

@@ -40,7 +40,7 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency 'rake', '>= 12.3.3'
   spec.add_development_dependency 'rake-compiler'
   spec.add_development_dependency 'rdoc'
-  spec.add_development_dependency 'rubocop', '~> 1.36'
+  spec.add_development_dependency 'standardrb', "~> 1.0"
 
   # Testing
   spec.add_development_dependency 'climate_control'

data/lib/appmap/handler/rails/request_handler.rb

@@ -76,12 +76,18 @@ module AppMap
           class << self
             def build_from_invocation(parent_id, return_value, elapsed, response, event: HTTPServerResponse.new)
               event ||= HTTPServerResponse.new
-              event.status = response[:status] || response.status
+
+              status = response[:status] || response.status
+              event.status = status if valid_http_status?(status)
               event.headers = (response[:headers] || response.headers).dup
               AppMap::Event::MethodReturn.build_from_invocation parent_id, return_value, nil, elapsed: elapsed, event: event, parameter_schema: true
             end
           end
 
+          def self.valid_http_status?(status)
+            status.is_a?(Integer) && status >= 100 && status < 600
+          end
+
           def to_h
             super.tap do |h|
               h[:http_server_response] = {

data/lib/appmap/value_inspector.rb

@@ -7,7 +7,7 @@ module AppMap
 
     def detect_size(value)
       # Don't risk calling #size on things like data-access objects, which can and will issue queries for this information.
-      if value.is_a?(Array) || value.is_a?(Hash)
+      if (value.is_a?(Array) || value.is_a?(Hash)) && value.respond_to?(:size)
         value.size
       end
     end

data/lib/appmap/version.rb

@@ -3,7 +3,7 @@
 module AppMap
   URL = 'https://github.com/applandinc/appmap-ruby'
 
-  VERSION = '0.102.0'
+  VERSION = '0.102.2'
 
   APPMAP_FORMAT_VERSION = '1.12.0'
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: appmap
 version: !ruby/object:Gem::Version
-  version: 0.102.0
+  version: 0.102.2
 platform: ruby
 authors:
 - Kevin Gilpin
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2023-07-18 00:00:00.000000000 Z
+date: 2023-11-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: method_source
@@ -151,19 +151,19 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rubocop
+  name: standardrb
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.36'
+        version: '1.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.36'
+        version: '1.0'
 - !ruby/object:Gem::Dependency
   name: climate_control
   requirement: !ruby/object:Gem::Requirement
@@ -324,7 +324,6 @@ files:
 - ".nvmrc"
 - ".rbenv-gemsets"
 - ".releaserc.yml"
-- ".rubocop.yml"
 - ".ruby-version"
 - ".rufo"
 - ".travis.yml"

data/.rubocop.yml

@@ -1,52 +0,0 @@
-AllCops:
-  NewCops: enable
-
-# The gem is automatically pushed by CI
-Gemspec/RequireMFA:
-  Enabled: false
-
-Layout/ArrayAlignment:
-  EnforcedStyle: with_fixed_indentation
-
-Layout/CaseIndentation:
-  EnforcedStyle: end
-
-Layout/FirstArgumentIndentation:
-  EnforcedStyle: consistent
-
-Layout/FirstArrayElementIndentation:
-  EnforcedStyle: consistent
-
-Layout/SpaceInsideArrayLiteralBrackets:
-  Enabled: false
-
-# We have squiggly heredocs
-Layout/HeredocIndentation:
-  Enabled: false
-
-Layout/LineLength:
-  Max: 120
-
-Metrics/BlockLength:
-  AllowedMethods:
-  - describe
-  - it
-  - context
-
-Style/MultilineBlockChain:
-  Enabled: false
-
-Style/NumericPredicate:
-  Enabled: false
-
-Style/AndOr:
-  Enabled: false
-
-Style/HashEachMethods:
-  Enabled: true
-
-Style/HashTransformKeys:
-  Enabled: true
-
-Style/HashTransformValues:
-  Enabled: true