Fix empty or null DSN not disabling Sentry (#454)

Author: ste93cry

CHANGELOG.md

@@ -2,7 +2,8 @@
 
 ## Unreleased
 
-- Add `kernel.project_dir` to `prefixes` default value to trim paths to the project root (#434) 
+- Add `kernel.project_dir` to `prefixes` default value to trim paths to the project root (#434)
+- Fix `null`, `false` or empty value not disabling Sentry (#454)
 
 ## 4.0.1 (2021-01-26)
 

src/DependencyInjection/Configuration.php

@@ -29,12 +29,7 @@ public function getConfigTreeBuilder(): TreeBuilder
         $rootNode
             ->children()
                 ->scalarNode('dsn')
-                    ->beforeNormalization()
-                        ->ifTrue(static function ($value): bool {
-                            return empty($value) || (\is_string($value) && '' === trim($value));
-                        })
-                        ->thenUnset()
-                    ->end()
+                    ->info('If this value is not provided, the SDK will try to read it from the SENTRY_DSN environment variable. If that variable also does not exist, the SDK will just not send any events.')
                 ->end()
                 ->booleanNode('register_error_listener')->defaultTrue()->end()
                 ->arrayNode('options')

tests/DependencyInjection/Fixtures/php/empty_dsn.php

@@ -236,14 +236,6 @@ public function testClentIsCreatedFromOptions(): void
         $this->assertEquals($methodCalls[4][1][0]->getArgument(0), new Reference('sentry.client.options'));
     }
 
-    public function testEmptyDsnIsTreatedAsIfItWasUnset(): void
-    {
-        $container = $this->createContainerFromFixture('empty_dsn');
-        $optionsDefinition = $container->getDefinition('sentry.client.options');
-
-        $this->assertArrayNotHasKey('dsn', $optionsDefinition->getArgument(0));
-    }
-
     public function testErrorTypesOptionIsParsedFromStringToIntegerValue(): void
     {
         $container = $this->createContainerFromFixture('error_types');