search

govCMS / GovCMS7

Current stable release of the main Drupal 7 GovCMS distribution, with releases mirrored at https://www.drupal.org/project/govcms
https://www.govcms.gov.au/
GNU General Public License v2.0
113 stars 76 forks source link

Patch redis module to force a timeout #962

Closed seanhamlin closed 4 years ago

seanhamlin commented 4 years ago

See https://www.drupal.org/project/redis/issues/2854377 See https://git.drupalcode.org/project/redis/-/blob/7.x-3.x/lib/Redis/Client/PhpRedis.php#L14 See https://github.com/amazeeio/drupal-example/pull/70#issuecomment-611589542

The redis module should be patched with a timeout, to ensure that Drupal only waits a sensible amount of time to connect to Redis before marking it as down. By default, Drupal will wait an unlimited amount of time.

Drupal 8 already has this functionality in the scaffolding - see https://github.com/govCMS/govcms8lagoon/commit/a467b5f8edc4a745eda7e66bb545ae1cf3f845f5

seanhamlin commented 4 years ago

closing in favour of https://github.com/govCMS/govcmslagoon/issues/148