Revert "Make the request tracing enableable"

ste93cry · ste93cry · commit b2500fb71a7a · 2021-03-02T18:30:37.000+01:00
This reverts commit 39891a4.
diff --git a/phpstan-baseline.neon b/phpstan-baseline.neon
@@ -11,7 +11,7 @@ parameters:
 			path: src/DependencyInjection/Configuration.php
 
 		-
-			message: "#^Call to an undefined method Symfony\\\\Component\\\\Config\\\\Definition\\\\Builder\\\\NodeParentInterface\\:\\:arrayNode\\(\\)\\.$#"
+			message: "#^Call to an undefined method Symfony\\\\Component\\\\Config\\\\Definition\\\\Builder\\\\NodeParentInterface\\:\\:end\\(\\)\\.$#"
 			count: 1
 			path: src/DependencyInjection/Configuration.php
 
@@ -106,17 +106,17 @@ parameters:
 			path: src/aliases.php
 
 		-
-			message: "#^Class Symfony\\\\Component\\\\HttpKernel\\\\Event\\\\FilterControllerEvent not found\\.$#"
+			message: "#^Class Symfony\\\\Component\\\\HttpKernel\\\\Event\\\\FilterResponseEvent not found\\.$#"
 			count: 1
 			path: src/aliases.php
 
 		-
-			message: "#^Class Symfony\\\\Component\\\\HttpKernel\\\\Event\\\\FilterResponseEvent not found\\.$#"
+			message: "#^Class Symfony\\\\Component\\\\HttpKernel\\\\Event\\\\PostResponseEvent not found\\.$#"
 			count: 1
 			path: src/aliases.php
 
 		-
-			message: "#^Class Symfony\\\\Component\\\\HttpKernel\\\\Event\\\\PostResponseEvent not found\\.$#"
+			message: "#^Class Symfony\\\\Component\\\\HttpKernel\\\\Event\\\\FilterControllerEvent not found\\.$#"
 			count: 1
 			path: src/aliases.php
 
diff --git a/src/DependencyInjection/Configuration.php b/src/DependencyInjection/Configuration.php
@@ -153,9 +153,6 @@ private function addDistributedTracingSection(ArrayNodeDefinition $rootNode): vo
                 ->arrayNode('tracing')
                     ->addDefaultsIfNotSet()
                     ->children()
-                        ->arrayNode('request')
-                            ->canBeEnabled()
-                        ->end()
                         ->arrayNode('dbal')
                             ->canBeEnabled()
                             ->fixXmlConfig('connection')
diff --git a/src/DependencyInjection/SentryExtension.php b/src/DependencyInjection/SentryExtension.php
@@ -16,8 +16,6 @@
 use Sentry\Options;
 use Sentry\SentryBundle\EventListener\ErrorListener;
 use Sentry\SentryBundle\EventListener\MessengerListener;
-use Sentry\SentryBundle\EventListener\TracingRequestListener;
-use Sentry\SentryBundle\EventListener\TracingSubRequestListener;
 use Sentry\SentryBundle\SentryBundle;
 use Sentry\SentryBundle\Tracing\Doctrine\DBAL\ConnectionConfigurator;
 use Sentry\SentryBundle\Tracing\Doctrine\DBAL\TracingDriverMiddleware;
@@ -64,7 +62,6 @@ protected function loadInternal(array $mergedConfig, ContainerBuilder $container
         $this->registerConfiguration($container, $mergedConfig);
         $this->registerErrorListenerConfiguration($container, $mergedConfig);
         $this->registerMessengerListenerConfiguration($container, $mergedConfig['messenger']);
-        $this->registerRequestTracingConfigurtion($container, $mergedConfig['tracing']);
         $this->registerDbalTracingConfiguration($container, $mergedConfig['tracing']);
         $this->registerTwigTracingConfiguration($container, $mergedConfig['tracing']);
     }
@@ -159,19 +156,6 @@ private function registerMessengerListenerConfiguration(ContainerBuilder $contai
         $container->getDefinition(MessengerListener::class)->setArgument(1, $config['capture_soft_fails']);
     }
 
-    /**
-     * @param array<string, mixed> $config
-     */
-    private function registerRequestTracingConfigurtion(ContainerBuilder $container, array $config): void
-    {
-        $isConfigEnabled = $this->isConfigEnabled($container, $config['request']);
-
-        if (!$isConfigEnabled) {
-            $container->removeDefinition(TracingRequestListener::class);
-            $container->removeDefinition(TracingSubRequestListener::class);
-        }
-    }
-
     /**
      * @param array<string, mixed> $config
      */
diff --git a/src/Resources/config/schema/sentry-1.0.xsd b/src/Resources/config/schema/sentry-1.0.xsd
@@ -83,16 +83,11 @@
 
     <xsd:complexType name="tracing">
         <xsd:choice maxOccurs="unbounded">
-            <xsd:element name="request" type="tracing-request" minOccurs="0" maxOccurs="1" />
             <xsd:element name="dbal" type="tracing-dbal" minOccurs="0" maxOccurs="1" />
             <xsd:element name="twig" type="tracing-twig" minOccurs="0" maxOccurs="1" />
         </xsd:choice>
     </xsd:complexType>
 
-    <xsd:complexType name="tracing-request">
-        <xsd:attribute name="enabled" type="xsd:boolean" />
-    </xsd:complexType>
-
     <xsd:complexType name="tracing-dbal">
         <xsd:sequence maxOccurs="unbounded">
             <xsd:element name="connection" type="xsd:string" minOccurs="0" maxOccurs="unbounded" />
diff --git a/tests/DependencyInjection/ConfigurationTest.php b/tests/DependencyInjection/ConfigurationTest.php
@@ -37,9 +37,6 @@ public function testProcessConfigurationWithDefaultConfiguration(): void
                 'capture_soft_fails' => true,
             ],
             'tracing' => [
-                'request' => [
-                    'enabled' => false,
-                ],
                 'dbal' => [
                     'enabled' => false,
                     'connections' => class_exists(DoctrineBundle::class) ? ['%doctrine.default_connection%'] : [],
diff --git a/tests/DependencyInjection/Fixtures/php/full.php b/tests/DependencyInjection/Fixtures/php/full.php
@@ -43,9 +43,6 @@
         'capture_soft_fails' => false,
     ],
     'tracing' => [
-        'request' => [
-            'enabled' => false,
-        ],
         'dbal' => [
             'enabled' => false,
             'connections' => ['default'],
diff --git a/tests/DependencyInjection/Fixtures/php/request_tracing_enabled.php b/tests/DependencyInjection/Fixtures/php/request_tracing_enabled.php
diff --git a/tests/DependencyInjection/Fixtures/xml/full.xml b/tests/DependencyInjection/Fixtures/xml/full.xml
@@ -38,7 +38,6 @@
         </sentry:options>
         <sentry:messenger enabled="true" capture-soft-fails="false" />
         <sentry:tracing>
-            <sentry:request enabled="false" />
             <sentry:dbal enabled="false">
                 <sentry:connection>default</sentry:connection>
             </sentry:dbal>
diff --git a/tests/DependencyInjection/Fixtures/xml/request_tracing_enabled.xml b/tests/DependencyInjection/Fixtures/xml/request_tracing_enabled.xml
diff --git a/tests/DependencyInjection/Fixtures/yml/full.yml b/tests/DependencyInjection/Fixtures/yml/full.yml
@@ -38,8 +38,6 @@ sentry:
         enabled: true
         capture_soft_fails: false
     tracing:
-        request:
-            enabled: false
         dbal:
             enabled: false
             connections:
diff --git a/tests/DependencyInjection/Fixtures/yml/request_tracing_enabled.yml b/tests/DependencyInjection/Fixtures/yml/request_tracing_enabled.yml
diff --git a/tests/DependencyInjection/SentryExtensionTest.php b/tests/DependencyInjection/SentryExtensionTest.php
@@ -16,8 +16,6 @@
 use Sentry\SentryBundle\EventListener\MessengerListener;
 use Sentry\SentryBundle\EventListener\RequestListener;
 use Sentry\SentryBundle\EventListener\SubRequestListener;
-use Sentry\SentryBundle\EventListener\TracingRequestListener;
-use Sentry\SentryBundle\EventListener\TracingSubRequestListener;
 use Sentry\SentryBundle\SentryBundle;
 use Sentry\SentryBundle\Tracing\Doctrine\DBAL\ConnectionConfigurator;
 use Sentry\SentryBundle\Tracing\Doctrine\DBAL\TracingDriverMiddleware;
@@ -267,22 +265,6 @@ public function testIgnoreErrorsIntegrationIsNotAddedTwiceIfAlreadyConfigured():
         $this->assertSame(1, $ignoreErrorsIntegrationsCount);
     }
 
-    public function testRequestTracingEventListenersAreConfiguredWhenRequestTracingIsEnabled(): void
-    {
-        $container = $this->createContainerFromFixture('request_tracing_enabled');
-
-        $this->assertTrue($container->hasDefinition(TracingRequestListener::class));
-        $this->assertTrue($container->hasDefinition(TracingSubRequestListener::class));
-    }
-
-    public function testRequestTracingEventListenersAreRemovedWhenRequestTracingIsDisabled(): void
-    {
-        $container = $this->createContainerFromFixture('full');
-
-        $this->assertFalse($container->hasDefinition(TracingRequestListener::class));
-        $this->assertFalse($container->hasDefinition(TracingSubRequestListener::class));
-    }
-
     public function testTracingDriverMiddlewareIsConfiguredWhenDbalTracingIsEnable(): void
     {
         if (!class_exists(DoctrineBundle::class)) {

Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,7 @@ parameters:`
`11`	`11`	`path: src/DependencyInjection/Configuration.php`
`12`	`12`
`13`	`13`	`-`
`14`		`- message: "#^Call to an undefined method Symfony\\\\Component\\\\Config\\\\Definition\\\\Builder\\\\NodeParentInterface\\:\\:arrayNode\\(\\)\\.$#"`
	`14`	`+ message: "#^Call to an undefined method Symfony\\\\Component\\\\Config\\\\Definition\\\\Builder\\\\NodeParentInterface\\:\\:end\\(\\)\\.$#"`
`15`	`15`	`count: 1`
`16`	`16`	`path: src/DependencyInjection/Configuration.php`
`17`	`17`
`@@ -106,17 +106,17 @@ parameters:`
`106`	`106`	`path: src/aliases.php`
`107`	`107`
`108`	`108`	`-`
`109`		`- message: "#^Class Symfony\\\\Component\\\\HttpKernel\\\\Event\\\\FilterControllerEvent not found\\.$#"`
	`109`	`+ message: "#^Class Symfony\\\\Component\\\\HttpKernel\\\\Event\\\\FilterResponseEvent not found\\.$#"`
`110`	`110`	`count: 1`
`111`	`111`	`path: src/aliases.php`
`112`	`112`
`113`	`113`	`-`
`114`		`- message: "#^Class Symfony\\\\Component\\\\HttpKernel\\\\Event\\\\FilterResponseEvent not found\\.$#"`
	`114`	`+ message: "#^Class Symfony\\\\Component\\\\HttpKernel\\\\Event\\\\PostResponseEvent not found\\.$#"`
`115`	`115`	`count: 1`
`116`	`116`	`path: src/aliases.php`
`117`	`117`
`118`	`118`	`-`
`119`		`- message: "#^Class Symfony\\\\Component\\\\HttpKernel\\\\Event\\\\PostResponseEvent not found\\.$#"`
	`119`	`+ message: "#^Class Symfony\\\\Component\\\\HttpKernel\\\\Event\\\\FilterControllerEvent not found\\.$#"`
`120`	`120`	`count: 1`
`121`	`121`	`path: src/aliases.php`
`122`	`122`