emit-rs / emit

Developer-first diagnostics for Rust applications
https://emit-rs.io
Apache License 2.0
128 stars 2 forks source link

OTLP exporter with gRPC: endpoint requires HTTP/1 #87

Closed KodrAus closed 2 months ago

KodrAus commented 2 months ago
warn span OTLP batch of 1 events to http://localhost:5341/opentelemetry.proto.collector.logs.v1.LogsService/Export failed: failed to send HTTP request
  err: failed to send HTTP request
  caused by: http2 error
  caused by: connection error received: endpoint requires HTTP/1.1

Looks like we need to handle this. We may be best off trying to establish a HTTP/1.1 connection and then upgrade.

KodrAus commented 2 months ago

This may be an issue with the target service not supporting gRPC over plain HTTP, but will leave open for now.

KodrAus commented 2 months ago

Since gRPC requires HTTP2 there's no expectation that a gRPC server can handle the HTTP1 upgrade flow and it seems many don't so I'll close this one now as something we won't do. It could be good to revisit in the future though.