Skip to content

Commit e5772a9

Browse files
minor #61394 [Messenger] Show package installation hint also for amqps (shyim)
This PR was merged into the 6.4 branch. Discussion ---------- [Messenger] Show package installation hint also for amqps | Q | A | ------------- | --- | Branch? | 6.4 | Bug fix? | yes? | New feature? | no | Deprecations? | no | Issues | | License | MIT A customer was confused that he could not use `amqps`, but he did forget to install the package. Adjusted the check to show that package also for `amqps` urls Commits ------- f951ffc [Messenger] Show package installation hint also for amqps
2 parents 04a6185 + f951ffc commit e5772a9

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/Symfony/Component/Messenger/Transport/TransportFactory.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ public function createTransport(#[\SensitiveParameter] string $dsn, array $optio
4141

4242
// Help the user to select Symfony packages based on protocol.
4343
$packageSuggestion = '';
44-
if (str_starts_with($dsn, 'amqp://')) {
44+
if (str_starts_with($dsn, 'amqp://') || str_starts_with($dsn, 'amqps://')) {
4545
$packageSuggestion = ' Run "composer require symfony/amqp-messenger" to install AMQP transport.';
4646
} elseif (str_starts_with($dsn, 'doctrine://')) {
4747
$packageSuggestion = ' Run "composer require symfony/doctrine-messenger" to install Doctrine transport.';

0 commit comments

Comments
 (0)