Remove full DSNs from exception messages

symfony · Nov 3, 2023 · ba72f72 · ba72f72
1 parent 8560dc5
commit ba72f72
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 7 deletions.
diff --git a/Provider/AbstractProviderFactory.php b/Provider/AbstractProviderFactory.php
@@ -28,7 +28,7 @@ abstract protected function getSupportedSchemes(): array;
     protected function getUser(Dsn $dsn): string
     {
         if (null === $user = $dsn->getUser()) {
-            throw new IncompleteDsnException('User is not set.', $dsn->getOriginalDsn());
+            throw new IncompleteDsnException('User is not set.', $dsn->getScheme().'://'.$dsn->getHost());
         }
 
         return $user;

diff --git a/Provider/Dsn.php b/Provider/Dsn.php
@@ -34,16 +34,16 @@ public function __construct(string $dsn)
         $this->originalDsn = $dsn;
 
         if (false === $parsedDsn = parse_url($dsn)) {
-            throw new InvalidArgumentException(sprintf('The "%s" translation provider DSN is invalid.', $dsn));
+            throw new InvalidArgumentException('The translation provider DSN is invalid.');
         }
 
         if (!isset($parsedDsn['scheme'])) {
-            throw new InvalidArgumentException(sprintf('The "%s" translation provider DSN must contain a scheme.', $dsn));
+            throw new InvalidArgumentException('The translation provider DSN must contain a scheme.');
         }
         $this->scheme = $parsedDsn['scheme'];
 
         if (!isset($parsedDsn['host'])) {
-            throw new InvalidArgumentException(sprintf('The "%s" translation provider DSN must contain a host (use "default" by default).', $dsn));
+            throw new InvalidArgumentException('The translation provider DSN must contain a host (use "default" by default).');
         }
         $this->host = $parsedDsn['host'];
 

diff --git a/Tests/Provider/DsnTest.php b/Tests/Provider/DsnTest.php
@@ -155,17 +155,17 @@ public static function invalidDsnProvider(): iterable
     {
         yield [
             'some://',
-            'The "some://" translation provider DSN is invalid.',
+            'The translation provider DSN is invalid.',
         ];
 
         yield [
             '//loco',
-            'The "//loco" translation provider DSN must contain a scheme.',
+            'The translation provider DSN must contain a scheme.',
         ];
 
         yield [
             'file:///some/path',
-            'The "file:///some/path" translation provider DSN must contain a host (use "default" by default).',
+            'The translation provider DSN must contain a host (use "default" by default).',
         ];
     }