Remove full DSNs from exception messages

Author: nicolas-grekas

Tests/Transport/DsnTest.php

@@ -155,17 +155,17 @@ public static function invalidDsnProvider(): iterable
     {
         yield [
             'some://',
-            'The "some://" notifier DSN is invalid.',
+            'The notifier DSN is invalid.',
         ];
 
         yield [
             '//slack',
-            'The "//slack" notifier DSN must contain a scheme.',
+            'The notifier DSN must contain a scheme.',
         ];
 
         yield [
             'file:///some/path',
-            'The "file:///some/path" notifier DSN must contain a host (use "default" by default).',
+            'The notifier DSN must contain a host (use "default" by default).',
         ];
     }
 

Transport/AbstractTransportFactory.php

@@ -46,7 +46,7 @@ protected function getUser(Dsn $dsn): string
     {
         $user = $dsn->getUser();
         if (null === $user) {
-            throw new IncompleteDsnException('User is not set.', $dsn->getOriginalDsn());
+            throw new IncompleteDsnException('User is not set.', $dsn->getScheme().'://'.$dsn->getHost());
         }
 
         return $user;

Transport/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" notifier DSN is invalid.', $dsn));
+            throw new InvalidArgumentException('The notifier DSN is invalid.');
         }
 
         if (!isset($parsedDsn['scheme'])) {
-            throw new InvalidArgumentException(sprintf('The "%s" notifier DSN must contain a scheme.', $dsn));
+            throw new InvalidArgumentException('The notifier DSN must contain a scheme.');
         }
         $this->scheme = $parsedDsn['scheme'];
 
         if (!isset($parsedDsn['host'])) {
-            throw new InvalidArgumentException(sprintf('The "%s" notifier DSN must contain a host (use "default" by default).', $dsn));
+            throw new InvalidArgumentException('The notifier DSN must contain a host (use "default" by default).');
         }
         $this->host = $parsedDsn['host'];