Split shorten to be able to disable it by using max_length: ~

Author: allflame

src/DependencyInjection/JaegerExtension.php

@@ -27,7 +27,10 @@ public function load(array $configs, ContainerBuilder $container)
         }
 
         if ($this->isConfigEnabled($container, $config['name_generator'])) {
-            $container->setParameter('jaeger.name.max_length', $config['name_generator']['max_length']);
+            if ($config['name_generator']['max_length']) {
+                $loader->load('shorten.yml');
+                $container->setParameter('jaeger.name.max_length', $config['name_generator']['max_length']);
+            }
             foreach ($config['name_generator']['request'] as $item => $customGeneratorId) {
                 $regexp = \sprintf('/%s/', $item);
                 $shortenedGeneratorId = \sprintf('jaeger.name.generator.%s', $customGeneratorId);

src/Resources/services.yml

@@ -12,7 +12,6 @@ parameters:
   env(JAEGER_DEBUG_COOKIE): 'debug'
   env(JAEGER_SERVICE_NAME): '%service_name%'
   env(JAEGER_SPAN_BATCH): '16'
-  jaeger.name.max_length: 64
 
 services:
   spl.stack:
@@ -152,10 +151,6 @@ services:
       - '@spl.priority.queue'
   jaeger.name.generator:
     alias: 'jaeger.name.generator.chain'
-  jaeger.name.generator.shorten:
-    class: Jaeger\Symfony\Name\Generator\ShortenGeneratorDecorator
-    decorates: jaeger.name.generator
-    arguments: ['@jaeger.name.generator.shorten.inner', '%jaeger.name.max_length%']
   jaeger.span.handler.background:
     class: Jaeger\Symfony\Bridge\BackgroundSpanHandler
     arguments: ['@jaeger.tracer']

src/Resources/shorten.yml

@@ -0,0 +1,7 @@
+parameters:
+  jaeger.name.max_length: 64
+services:
+  jaeger.name.generator.shorten:
+    class: Jaeger\Symfony\Name\Generator\ShortenGeneratorDecorator
+    decorates: jaeger.name.generator
+    arguments: [ '@jaeger.name.generator.shorten.inner', '%jaeger.name.max_length%' ]