open-telemetry / opentelemetry-cpp

The OpenTelemetry C++ Client
https://opentelemetry.io/
Apache License 2.0
851 stars 405 forks source link

[EXPORTER] exporter endpoint doesn't support ipv6 #3073

Open marcalff opened 6 days ago

marcalff commented 6 days ago

Discussed in https://github.com/open-telemetry/opentelemetry-cpp/discussions/3056

Originally posted by **qusijun** September 9, 2024 now, UrlParser can't parse ipv6 correctly , do we have the plan to support ipv6?
marcalff commented 6 days ago

The URL parser just looks for a : character in a url, it will be very confused when parsing IPv6 addresses.

https://github.com/open-telemetry/opentelemetry-cpp/blob/main/ext/include/opentelemetry/ext/http/common/url_parser.h