[Notifier] Change return type

UPGRADE-5.3.md

@@ -23,6 +23,11 @@ HttpKernel
 
  * Marked the class `Symfony\Component\HttpKernel\EventListener\DebugHandlersListener` as internal
 
+Notifier
+-------
+
+* Changed the return type of `Symfony\Component\Notifier\Transport\AbstractTransportFactory::getEndpoint()` from `?string` to `string`
+
 PhpunitBridge
 -------------
 

src/Symfony/Component/Notifier/Bridge/Infobip/InfobipTransport.php

@@ -85,7 +85,7 @@ final class InfobipTransport extends AbstractTransport
         return new SentMessage($message, (string) $this);
     }
 
-    protected function getEndpoint(): ?string
+    protected function getEndpoint(): string
     {
         return $this->host.($this->port ? ':'.$this->port : '');
     }

src/Symfony/Component/Notifier/Bridge/Mattermost/MattermostTransport.php

@@ -85,7 +85,7 @@ final class MattermostTransport extends AbstractTransport
         return $sentMessage;
     }
 
-    protected function getEndpoint(): ?string
+    protected function getEndpoint(): string
     {
         return rtrim($this->host.($this->port ? ':'.$this->port : '').($this->path ?? ''), '/');
     }

src/Symfony/Component/Notifier/CHANGELOG.md

@@ -5,6 +5,7 @@ CHANGELOG
 -----
 
  * The component is not marked as `@experimental` anymore
+* [BC BREAK] Changed the return type of `AbstractTransportFactory::getEndpoint()` from `?string` to `string`
 
 5.2.0
 -----

src/Symfony/Component/Notifier/Transport/AbstractTransport.php

@@ -79,7 +79,7 @@ abstract class AbstractTransport implements TransportInterface
 
     abstract protected function doSend(MessageInterface $message): SentMessage;
 
-    protected function getEndpoint(): ?string
+    protected function getEndpoint(): string
     {
         return ($this->host ?: $this->getDefaultHost()).($this->port ? ':'.$this->port : '');
     }