Update exporters/otlp/otlplog/otlploghttp/config.go

tongoss · dmathieu · tongoss · commit c6f0fa374eed · 2025-03-06T08:06:36.000-05:00
Apply the suggested changes

Co-authored-by: Damien Mathieu &lt;42@dmathieu.com&gt;
diff --git a/exporters/otlp/otlplog/otlploghttp/config.go b/exporters/otlp/otlplog/otlploghttp/config.go
@@ -556,7 +556,6 @@ func convHeaders(s string) (map[string]string, error) {
 		// Only decode the value.
 		escVal, e := url.PathUnescape(rawVal)
 		if e != nil {
-			global.Error(e, "escape header value", "value", rawVal)
 			err = errors.Join(err, fmt.Errorf("invalid header value: %s", rawVal))
 			continue
 		}

Original file line number	Diff line number	Diff line change
`@@ -556,7 +556,6 @@ func convHeaders(s string) (map[string]string, error) {`
`556`	`556`	`// Only decode the value.`
`557`	`557`	`escVal, e := url.PathUnescape(rawVal)`
`558`	`558`	`if e != nil {`
`559`		`- global.Error(e, "escape header value", "value", rawVal)`
`560`	`559`	`err = errors.Join(err, fmt.Errorf("invalid header value: %s", rawVal))`
`561`	`560`	`continue`
`562`	`561`	`}`