opentelemetry-exporter-otlp: http_connection does not handle ipv6 host

[acarella1]

The OT library uses host from URI which fails to unwrap ipv6 brackets properly. Fix is to update this to use hostname instead, which will return a plain ipv6 address which is what Net::HTTP expects as the first argument.

opentelemetry-ruby/exporter/otlp/lib/opentelemetry/exporter/otlp/exporter.rb

Line 107 in 75a4885

http = Net::HTTP.new(uri.host, uri.port)

Share details about your runtime

Operating system details: FROM ruby:3.3.5-bookworm
RUBY_ENGINE: "ruby"
RUBY_VERSION: "3.3.5"
RUBY_DESCRIPTION: "ruby 3.3.5 (2024-09-03 revision ef084cc8f4) [aarch64-linux]"

Share a simplified reproduction if possible

failure.rb

require 'net/http'
uri = URI('http://[::1]:4318')
http = Net::HTTP.new(uri.host, uri.port)
http.start

(hard error) getaddrinfo: Name or service not known (Socket::ResolutionError)

success.rb

require 'net/http'
uri = URI('http://[::1]:4318')
http = Net::HTTP.new(uri.hostname, uri.port)
http.start

(would be a success if something were on the port) Connection refused - connect(2) for "::1" port 4318 (Errno::ECONNREFUSED)

[kaylareopelle]

Hi @acarella1, thanks for reporting this bug! It seems like you have a good idea about what's needed to fix the problem. Would you be interested in submitting a PR?

[acarella1]

Hi @kaylareopelle,
In my opinion Net::HTTP should tolerate this input, however it does not, even in the latest version of Ruby.
I will submit a PR in some time, thank you.

[github-actions]

👋 This issue has been marked as stale because it has been open with no activity. You can: comment on the issue or remove the stale label to hold stale off for a while, add the keep label to hold stale off permanently, or do nothing. If you do nothing this issue will be closed eventually by the stale bot.

[create2000]

Hello @kaylareopelle -- My name is Anthony Onah and I am an Outreachy Internship applicant. Can I be assigned this issue? If it's still open to being worked on.

[acarella1]

Hi @create2000 I'll be taking care of this one in some time, thank you