bug #42035 [Messenger] Fix use_notify default value for PostgreSqlConnection (tgalopin)

tgalopin · tgalopin · commit 9fdc06bf7294 · 2021-07-09T10:41:40.000+02:00
This PR was merged into the 5.2 branch. Discussion ---------- [Messenger] Fix use_notify default value for PostgreSqlConnection | Q | A | ------------- | --- | Branch? | 5.2 | Bug fix? | yes | New feature? | no | Deprecations? | no | Tickets | - | License | MIT | Doc PR | - I tried to use the use_notify in my DSN with Doctrine and stumbled upon an error. I investigated and from what I see, the option isn't allowed in a DSN due to this line missing in the PostgreSqlConnection class. Commits ------- b2aad4f Fix use_notify default value for PostgreSqlConnection
diff --git a/src/Symfony/Component/Messenger/Bridge/Doctrine/Transport/PostgreSqlConnection.php b/src/Symfony/Component/Messenger/Bridge/Doctrine/Transport/PostgreSqlConnection.php
@@ -28,6 +28,7 @@ final class PostgreSqlConnection extends Connection
      * * get_notify_timeout: The length of time to wait for a response when calling PDO::pgsqlGetNotify, in milliseconds. Default: 0.
      */
     protected const DEFAULT_OPTIONS = parent::DEFAULT_OPTIONS + [
+        'use_notify' => true,
         'check_delayed_interval' => 60000,
         'get_notify_timeout' => 0,
     ];
