remove usage of OpenStruct in DI

DataDog · Jan 9, 2025 · df02a87 · df02a87
1 parent aff0f7c
commit df02a87
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 2 deletions.
diff --git a/integration/apps/hanami/Gemfile b/integration/apps/hanami/Gemfile
@@ -2,6 +2,7 @@ require 'datadog/demo_env'
 
 source 'https://rubygems.org'
 
+gem 'ostruct'
 gem 'rake'
 gem 'hanami',       '~> 1.3'
 gem 'hanami-model', '~> 1.3'

diff --git a/lib/datadog/di/transport.rb b/lib/datadog/di/transport.rb
@@ -62,7 +62,8 @@ def send_input(payload)
 
       def send_request(desc, **options)
         # steep:ignore:start
-        env = OpenStruct.new(**options)
+        #env = OpenStruct.new(**options)
+        env = Core::Transport::HTTP::Env.new(nil, options)
         # steep:ignore:end
         response = client.post(env)
         unless response.ok?

diff --git a/spec/datadog/di/integration/everything_from_remote_config_spec.rb b/spec/datadog/di/integration/everything_from_remote_config_spec.rb
@@ -217,7 +217,7 @@ def do_rc
     # Events can be batched, meaning +post+ could be called once or twice
     # depending on how threads are scheduled by the VM.
     expect(component.transport.send(:client)).to receive(:post).at_least(:once) do |env|
-      expect(env).to be_a(OpenStruct)
+      #expect(env).to be_a(OpenStruct)
       notify_payload = if env.path == '/debugger/v1/diagnostics'
         JSON.parse(env.form.fetch('event').io.read, symbolize_names: true)
       else