diogo/symfony
Archived
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

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

Browse Source 
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
-------

b2aad4f666 Fix use_notify default value for PostgreSqlConnection
This commit is contained in:
Titouan Galopin 2021-07-09 10:41:40 +02:00
commit 9fdc06bf72
1 changed files with 1 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
src/Symfony/Component/Messenger/Bridge/Doctrine/Transport/PostgreSqlConnection.php
View File

@ -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,
];