betagouv/rdv-service-public

SSL_read: shutdown while in init (OpenSSL::SSL::SSLError)

Closed this issue · 1 comments

francois-ferrandis commented

Nous avons entre 0 et 15 occurrences de cette erreur par jour :
https://sentry.incubateur.net/organizations/betagouv/issues/93337

J'ai trouvé des infos ici : redis/redis-rb#1174
Il semblerait que le problème soit potentiellement résolu dans une version plus récente de la gem redis. En effet, la gem redis se met à partir de sa version 5 à utilise redis-client, une gem bas niveau qui gère la connexion à Redis, et qui pourrait retry plutôt que crasher en cas de "shutdown while in init" (depuis cette PR).

Est-ce une bonne idée de mettre à jour la gem ?

Nous avions déjà mis à jour vers cette version dans #4144, mais nous avions eu des soucis de timeout car la version 5 de redis passe le timeout par défaut de 5 secondes à 1 secondes. Cette valeur de timeout était trop courte et causait des soucis (#4151) car nous n'utilisions pas de pool de connexion. Nous avons depuis mis en place une pool de connexion (#4199), donc nous devrions avoir moins de "connection timeout".

Je propose donc de mettre à jour la gem redis en version 5 pour corriger le crash de cette issue, tout en explicitant un timeout par défaut à 5 seconde pour rester iso sur la config et ne pas rencontrer les crashes qui nous ont poussé à ne pas mettre à jour la gem à l'époque (#4151).