opentelemetry-instrumentation-rack 0.19.3 → 0.20.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 84f8534ada7c9bcadbd3b67d895ac3735ae76e76816a3fb9d170b8bc806d2610
-  data.tar.gz: a02745f74e4483eab099d93ad1bf12d051530da2ddc06e8fe26cae2ec23c8d62
+  metadata.gz: ac4a58d7625183e4c21d3e46db230f33a7cbffba733cfce9458750c1eaf0cd1f
+  data.tar.gz: 9d2328bea5c516b009b82d8fcfd4bba1ccce2a106fa1bdfae353053264b5794f
 SHA512:
-  metadata.gz: 29a49222b833daf6e04625882aba90e4f343cccb7a1328b1e7e84e8c613ad4fe1d7db84b3053cf322ac71e9b15285d07e9a980a653fe966150f4fc6b74df14da
-  data.tar.gz: 92f2cd31fb3a6933b4f2c4bdc523dc99863a9bd01677b1b6280d089e4a3aac1f6d4138a0fe50294a5953be55cab0e52b16452fb38b2d9016ff51391b374939f7
+  metadata.gz: 64387f52f98e5fb157640f3a5ec1fe6a47a2df397b8c28436f15dbcfadb1a35dc82f3f3f1102d1a81fa3d97088a8a7754c3d12a49375c265965f0c940f28876f
+  data.tar.gz: d7ce5b0ad0453f929abf40b0ccca1634270b3b2048f6e8eff9de6d301a82344120963f99e9f811156abda172b0a10c50380f9f8094f7b8b9e3e65dcd51744297

data/CHANGELOG.md

@@ -1,5 +1,25 @@
 # Release History: opentelemetry-instrumentation-rack
 
+### v0.20.2 / 2022-05-02
+
+* FIXED: Update server instrumentation to not reflect 400 status as error 
+
+### v0.20.1 / 2021-12-01
+
+* FIXED: [Instruentation Rack] Log content type http header 
+* FIXED: Use monotonic clock where possible 
+* FIXED: Rack to stop using api env getter 
+
+### v0.20.0 / 2021-10-06
+
+* FIXED: Prevent high cardinality rack span name as a default [#973](https://github.com/open-telemetry/opentelemetry-ruby/pull/973)
+
+The default was to set the span name as the path of the request, we have
+corrected this as it was not adhering to the spec requirement using low
+cardinality span names.  You can restore the previous behaviour of high
+cardinality span names by passing in a url quantization function that
+forwards the uri path.  More details on this is available in the readme.
+
 ### v0.19.3 / 2021-09-29
 
 * (No significant changes)

data/README.md

@@ -29,6 +29,25 @@ OpenTelemetry::SDK.configure do |c|
   c.use_all
 end
 ```
+## Controlling span name cardinality
+
+By default we will set the rack span name to match the format "HTTP #{method}" (ie. HTTP GET). There are different ways to control span names with this instrumentation.
+
+### Enriching rack spans
+
+We surface a hook to easily retrieve the rack span within the context of a request so that you can add information to or rename your server span.
+
+This is how the rails controller instrumentation is able to rename the span names to match the controller and action that process the request. See https://github.com/open-telemetry/opentelemetry-ruby/blob/f6fb025bef69f839078748f56516ce38c7d51eb8/instrumentation/action_pack/lib/opentelemetry/instrumentation/action_pack/patches/action_controller/metal.rb#L15-L16 for an example.
+
+### High cardinality example
+
+You can pass in an url quantization lambda that simply uses the URL path, the result is you will end up with high cardinality span names, however this may be acceptable in your deployment and is easy configurable using the following example.
+
+```ruby
+OpenTelemetry::SDK.configure do |c|
+  c.use 'OpenTelemetry::Instrumentation::Rack', { url_quantization: ->(path, _env) { path.to_s } }
+end
+```
 
 ## Examples
 
@@ -50,4 +69,4 @@ The `opentelemetry-instrumentation-rack` gem is distributed under the Apache 2.0
 [license-github]: https://github.com/open-telemetry/opentelemetry-ruby/blob/main/LICENSE
 [ruby-sig]: https://github.com/open-telemetry/community#ruby-sig
 [community-meetings]: https://github.com/open-telemetry/community#community-meetings
-[discussions-url]: https://github.com/open-telemetry/opentelemetry-ruby/discussions
+[discussions-url]: https://github.com/open-telemetry/opentelemetry-ruby/discussions

data/lib/opentelemetry/instrumentation/rack/middlewares/tracer_middleware.rb

@@ -18,7 +18,13 @@ module OpenTelemetry
           class << self
             def allowed_rack_request_headers
               @allowed_rack_request_headers ||= Array(config[:allowed_request_headers]).each_with_object({}) do |header, memo|
-                memo["HTTP_#{header.to_s.upcase.gsub(/[-\s]/, '_')}"] = build_attribute_name('http.request.headers.', header)
+                key = header.to_s.upcase.gsub(/[-\s]/, '_')
+                case key
+                when 'CONTENT_TYPE', 'CONTENT_LENGTH'
+                  memo[key] = build_attribute_name('http.request.headers.', header)
+                else
+                  memo["HTTP_#{key}"] = build_attribute_name('http.request.headers.', header)
+                end
               end
             end
 
@@ -58,10 +64,11 @@ module OpenTelemetry
                 return @app.call(env)
               end
             end
+
             original_env = env.dup
             extracted_context = OpenTelemetry.propagation.extract(
               env,
-              getter: OpenTelemetry::Context::Propagation.rack_env_getter
+              getter: OpenTelemetry::Common::Propagation.rack_env_getter
             )
             frontend_context = create_frontend_span(env, extracted_context)
 
@@ -100,9 +107,7 @@ module OpenTelemetry
 
             span = tracer.start_span('http_server.proxy',
                                      with_parent: extracted_context,
-                                     attributes: {
-                                       'start_time' => request_start_time.to_f
-                                     },
+                                     start_timestamp: request_start_time,
                                      kind: :server)
 
             OpenTelemetry::Trace.context_with_span(span, parent_context: extracted_context)
@@ -141,12 +146,12 @@ module OpenTelemetry
             if (implementation = config[:url_quantization])
               implementation.call(request_uri_or_path_info, env)
             else
-              request_uri_or_path_info
+              "HTTP #{env['REQUEST_METHOD']}"
             end
           end
 
           def set_attributes_after_request(span, status, headers, _response)
-            span.status = OpenTelemetry::Trace::Status.error unless (100..399).include?(status.to_i)
+            span.status = OpenTelemetry::Trace::Status.error unless (100..499).include?(status.to_i)
             span.set_attribute('http.status_code', status)
 
             # NOTE: if data is available, it would be good to do this:

data/lib/opentelemetry/instrumentation/rack/version.rb

@@ -7,7 +7,7 @@
 module OpenTelemetry
   module Instrumentation
     module Rack
-      VERSION = '0.19.3'
+      VERSION = '0.20.2'
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: opentelemetry-instrumentation-rack
 version: !ruby/object:Gem::Version
-  version: 0.19.3
+  version: 0.20.2
 platform: ruby
 authors:
 - OpenTelemetry Authors
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-09-30 00:00:00.000000000 Z
+date: 2022-05-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: opentelemetry-api
@@ -24,20 +24,34 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.0'
+- !ruby/object:Gem::Dependency
+  name: opentelemetry-common
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.19.3
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.19.3
 - !ruby/object:Gem::Dependency
   name: opentelemetry-instrumentation-base
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.18.3
+        version: 0.20.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.18.3
+        version: 0.20.0
 - !ruby/object:Gem::Dependency
   name: appraisal
   requirement: !ruby/object:Gem::Requirement
@@ -228,10 +242,10 @@ homepage: https://github.com/open-telemetry/opentelemetry-ruby
 licenses:
 - Apache-2.0
 metadata:
-  changelog_uri: https://open-telemetry.github.io/opentelemetry-ruby/opentelemetry-instrumentation-rack/v0.19.3/file.CHANGELOG.html
+  changelog_uri: https://open-telemetry.github.io/opentelemetry-ruby/opentelemetry-instrumentation-rack/v0.20.2/file.CHANGELOG.html
   source_code_uri: https://github.com/open-telemetry/opentelemetry-ruby/tree/main/instrumentation/rack
   bug_tracker_uri: https://github.com/open-telemetry/opentelemetry-ruby/issues
-  documentation_uri: https://open-telemetry.github.io/opentelemetry-ruby/opentelemetry-instrumentation-rack/v0.19.3
+  documentation_uri: https://open-telemetry.github.io/opentelemetry-ruby/opentelemetry-instrumentation-rack/v0.20.2
 post_install_message: 
 rdoc_options: []
 require_paths: