Merge pull request #39 from NthPortal/no-send-redacted

Don't create redacted header attributes
http4s · Jan 2, 2024 · 21125a6 · 21125a6
2 parents 684673c + 732dcfe
commit 21125a6
Showing 1 changed file with 4 additions and 6 deletions.
diff --git a/core/src/main/scala/org/http4s/otel4s/middleware/HttpAttributes.scala b/core/src/main/scala/org/http4s/otel4s/middleware/HttpAttributes.scala
@@ -74,8 +74,6 @@ object HttpAttributes {
   }
 
   object Headers {
-    private final val Redacted = List("<REDACTED>")
-
     private[this] def unsafeGeneric(
         redactedHeaders: Headers,
         allowedHeaders: Set[CIString],
@@ -84,10 +82,10 @@ object HttpAttributes {
       redactedHeaders.headers
         .groupMap(_.name)(_.value)
         .view
-        .map { case (name, list) =>
-          val key = s"http.$messageType.header.${name.toString.toLowerCase(Locale.ROOT)}"
-          if (allowedHeaders.contains(name)) Attribute(key, list)
-          else Attribute(key, Redacted)
+        .collect {
+          case (name, values) if allowedHeaders.contains(name) =>
+            val key = s"http.$messageType.header.${name.toString.toLowerCase(Locale.ROOT)}"
+            Attribute(key, values)
         }
         .toList