-
Notifications
You must be signed in to change notification settings - Fork 401
Closed
Labels
feature-requestA request for a new feature or change to an existing oneA request for a new feature or change to an existing oneintegrationsInvolves tracing integrationsInvolves tracing integrations
Description
The Net::HTTP Ruby library can be installed and upgraded separately from the Ruby standard library since Ruby 3.0.0.
This means that the http integration's minimum supported version is now misleading:
dd-trace-rb/lib/datadog/tracing/contrib/http/integration.rb
Lines 21 to 27 in df224ac
| MINIMUM_VERSION = DDTrace::VERSION::MINIMUM_RUBY_VERSION | |
| # @public_api Changing the integration name or integration options can cause breaking changes | |
| register_as :http, auto_patch: true | |
| def self.version | |
| Gem::Version.new(RUBY_VERSION) |
I has currently has its minimum version set to the current Ruby version.
We should correctly mark which version of the net-http gem we support: rubygems.org/gems/net-http.
Also, if possible, we should rename the integration to net-http. This can cause breaking configuration changes when implemented (e.g. c.tracing.instrument :http will likely be changed).
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
feature-requestA request for a new feature or change to an existing oneA request for a new feature or change to an existing oneintegrationsInvolves tracing integrationsInvolves tracing integrations