fix(DependencyInjection/Configuration): normalize headers format according to PSR HTTP transport (#167)

Author: klkvsk

src/DependencyInjection/Configuration.php

@@ -587,10 +587,12 @@ private function getOtlpExportersOptionsNode(): ArrayNodeDefinition
                     ->values(array_map(fn (OtlpExporterCompressionEnum $format) => $format->value, OtlpExporterCompressionEnum::cases()))
                 ->end()
                 ->arrayNode('headers')
-                    ->arrayPrototype()
-                        ->children()
-                            ->scalarNode('name')->isRequired()->cannotBeEmpty()->end()
-                            ->scalarNode('value')->isRequired()->cannotBeEmpty()->end()
+                    ->useAttributeAsKey('name')
+                    ->normalizeKeys(false)
+                    ->variablePrototype()
+                        ->beforeNormalization()
+                            ->ifArray()
+                            ->then(static fn ($v) => array_values($v))
                         ->end()
                     ->end()
                 ->end()

tests/Unit/DependencyInjection/ConfigurationTest.php

@@ -292,9 +292,7 @@ public function testReferenceConfiguration(): void
                             headers:
 
                                 # Prototype
-                                -
-                                    name:                 ~ # Required
-                                    value:                ~ # Required
+                                name:                 ~
                             timeout:              0.1
                             retry:                100
                             max:                  3
@@ -331,9 +329,7 @@ public function testReferenceConfiguration(): void
                             headers:
 
                                 # Prototype
-                                -
-                                    name:                 ~ # Required
-                                    value:                ~ # Required
+                                name:                 ~
                             timeout:              0.1
                             retry:                100
                             max:                  3
@@ -395,9 +391,7 @@ public function testReferenceConfiguration(): void
                             headers:
 
                                 # Prototype
-                                -
-                                    name:                 ~ # Required
-                                    value:                ~ # Required
+                                name:                 ~
                             timeout:              0.1
                             retry:                100
                             max:                  3

tests/Unit/DependencyInjection/OpenTelemetryLogsExtensionTest.php

@@ -237,10 +237,7 @@ public static function exporters(): \Generator
                 'format' => 'json',
                 'compression' => 'gzip',
                 'headers' => [
-                    [
-                        'name' => 'X-Foo',
-                        'value' => 'Bar',
-                    ],
+                    'X-Foo' => 'Bar',
                 ],
                 'timeout' => 0.4,
                 'retry' => 500,

tests/Unit/DependencyInjection/OpenTelemetryMetricsExtensionTest.php

@@ -205,10 +205,7 @@ public static function exporters(): \Generator
                 'format' => 'json',
                 'compression' => 'gzip',
                 'headers' => [
-                    [
-                        'name' => 'X-Foo',
-                        'value' => 'Bar',
-                    ],
+                    'X-Foo' => 'Bar',
                 ],
                 'timeout' => 0.4,
                 'retry' => 500,

tests/Unit/DependencyInjection/OpenTelemetryTracesExtensionTest.php

@@ -215,10 +215,7 @@ public static function exporters(): \Generator
                 'format' => 'json',
                 'compression' => 'gzip',
                 'headers' => [
-                    [
-                        'name' => 'X-Foo',
-                        'value' => 'Bar',
-                    ],
+                    'X-Foo' => 'Bar',
                 ],
                 'timeout' => 0.4,
                 'retry' => 500,