From 290639a02e37f700c8fa7ed580ba092b0fe3256e Mon Sep 17 00:00:00 2001 From: Eduardo Silva Date: Mon, 17 Mar 2025 09:47:11 -0600 Subject: [PATCH] out_opentelemetry: switch config property 'http2' default to 'off' We have received many reports that Fluent Bit cannot send data to the OpenTelemetry Collector over a plain text communication, this is happening because we enabled http2 by default so when the Otel Collector receives the request (PRI * HTTP/2.0...) it drops the connection. This PR switches http2 default to off so users don't run into this problem. Signed-off-by: Eduardo Silva --- plugins/out_opentelemetry/opentelemetry.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/plugins/out_opentelemetry/opentelemetry.c b/plugins/out_opentelemetry/opentelemetry.c index caa74bfb7f6..9a4f335f29e 100644 --- a/plugins/out_opentelemetry/opentelemetry.c +++ b/plugins/out_opentelemetry/opentelemetry.c @@ -280,6 +280,7 @@ int opentelemetry_post(struct opentelemetry_context *ctx, if (request->protocol_version == HTTP_PROTOCOL_VERSION_20 && ctx->enable_grpc_flag) { + grpc_body = cfl_sds_create_size(body_len + 5); if (grpc_body == NULL) { @@ -730,7 +731,7 @@ static struct flb_config_map config_map[] = { "Adds a custom label to the metrics use format: 'add_label name value'" }, { - FLB_CONFIG_MAP_STR, "http2", "on", + FLB_CONFIG_MAP_STR, "http2", "off", 0, FLB_TRUE, offsetof(struct opentelemetry_context, enable_http2), "Enable, disable or force HTTP/2 usage. Accepted values : on, off, force" },