appmap 0.72.1 → 0.72.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d0dac1cd2861b93a4ec1865d66c561b11bb325263e4f2918192a455158c62429
-  data.tar.gz: 6565fc39df7835aa009fdba68890bb8879f771092b32c2c53f8140d3c2d69196
+  metadata.gz: 87322a2d5ba58dfc1fb69259af06c59077d3897f1e37e91e5a94dd2cbba1478b
+  data.tar.gz: ace21bbbd53fa487b27339e2af8fb16f4f8ec7b3ca7bbe218ba205b8ff7fc94b
 SHA512:
-  metadata.gz: 93ec8a28c7b4c6573b2be71c5a35fa41e6a3a4af560bc0f3e24d9bf22e43906916c9ecce04f1d0c2e4ee0feef3514dddc48a836df1d1f489180483c032a19261
-  data.tar.gz: 646801d2effd4ca81ded335a12295b86c12d60575096b6fa50375947682ea567f53bd54949525d1cf5c29bf36ba9c15e97d33dc3b207c65b5d262010e9cb4b8b
+  metadata.gz: 9eaebd3dbb1bf74880eeafc2379a78e73d747f4a41a507152c0d3840eefff1b4f8ae3c6cdecfb15bb08e44b5f3d19779b1c265746148ddb569715ace6c17a834
+  data.tar.gz: 4569886b26db0d49d06fd0bb649997fc29d9fea665c59495448e0b1669581757003f3034536392bf8e4e5cb16955ce12b4ca3e2ffd90cb8af8057a8edb4f093a

data/CHANGELOG.md

@@ -1,3 +1,10 @@
+## [0.72.2](https://github.com/applandinc/appmap-ruby/compare/v0.72.1...v0.72.2) (2022-02-11)
+
+
+### Bug Fixes
+
+* Ensure request headers includes all relevant fields ([e866f68](https://github.com/applandinc/appmap-ruby/commit/e866f686be974dfe29f706564217e4ee302bb55a))
+
 ## [0.72.1](https://github.com/applandinc/appmap-ruby/compare/v0.72.0...v0.72.1) (2022-01-31)
 
 

data/lib/appmap/util.rb

@@ -121,15 +121,21 @@ module AppMap
           blank?(headers) ? nil : headers
         end
 
-        # Rack prepends HTTP_ to all client-sent headers.
-
         if !env['rack.version']
           warn "Request headers does not contain rack.version. HTTP_ prefix is not expected."
           return finalize_headers.call(env.dup)
         end
 
+        # Rack prepends HTTP_ to all client-sent headers (except Content-Type and Content-Length?).
+        # Apparently, it's following the CGI spec in doing so.
+        # https://datatracker.ietf.org/doc/html/rfc3875#section-4.1.18
         matching_headers = env
-          .select { |k,v| k.start_with? 'HTTP_'}
+          .select { |k,v| k.start_with? 'HTTP_' }
+          .merge(
+            'CONTENT_TYPE' => env['CONTENT_TYPE'],
+            'CONTENT_LENGTH' => env['CONTENT_LENGTH'],
+            'AUTHORIZATION' => env['AUTHORIZATION']
+          )
           .reject { |k,v| blank?(v) }
           .each_with_object({}) do |kv, memo|
             key = kv[0].sub(/^HTTP_/, '').split('_').map(&:capitalize).join('-')

data/lib/appmap/version.rb

@@ -3,7 +3,7 @@
 module AppMap
   URL = 'https://github.com/applandinc/appmap-ruby'
 
-  VERSION = '0.72.1'
+  VERSION = '0.72.2'
 
   APPMAP_FORMAT_VERSION = '1.5.1'
 

data/spec/rails_recording_spec.rb

@@ -33,7 +33,8 @@ describe 'Rails' do
                 'http_server_request' => hash_including(
                   'request_method' => 'POST',
                   'normalized_path_info' => '/api/users',
-                  'path_info' => '/api/users'
+                  'path_info' => '/api/users',
+                  'headers' => hash_including('Content-Type' => 'application/x-www-form-urlencoded'),
                 ),
                 'message' => include(
                   hash_including(

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: appmap
 version: !ruby/object:Gem::Version
-  version: 0.72.1
+  version: 0.72.2
 platform: ruby
 authors:
 - Kevin Gilpin
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2022-01-31 00:00:00.000000000 Z
+date: 2022-02-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport