Project

General

Profile

Bug #21783

redéscendre le timeout à 3s mais permettre 3 essaies

Added by Benjamin Dauvergne over 1 year ago. Updated over 1 year ago.

Status:
Fermé
Priority:
Normal
Target version:
Start date:
11 Feb 2018
Due date:
% Done:

100%

Hors marché:
No
Patch proposed:
No
Planning:
No

Description

Si 3 erreurs consécutives, alors afficher une page d'erreur permettant à l'utilisateur de retenter.

0002-retry-HTTP-requests-3-times-fixes-21783.patch View (3.29 KB) Benjamin Dauvergne, 11 Feb 2018 10:12 PM

0001-tox.ini-remove-python-catchlog-which-is-obsolete.patch View (554 Bytes) Benjamin Dauvergne, 11 Feb 2018 10:12 PM

0002-retry-HTTP-requests-3-times-fixes-21783.patch View (4.49 KB) Benjamin Dauvergne, 11 Feb 2018 10:21 PM

0001-tox.ini-remove-python-catchlog-which-is-obsolete.patch View (554 Bytes) Benjamin Dauvergne, 11 Feb 2018 10:21 PM

Associated revisions

Revision c701e610 (diff)
Added by Benjamin Dauvergne over 1 year ago

retry HTTP requests 3 times (fixes #21783)

Retry is applied to access token request and user info requests (through
OAuth2Session). There is a small exponential backoff of 0.5 and 1s.

Also decrease log level of message for failure of retrieval of the
access token or the user info to the level WARNING, that's never been a
problem for the user, as he was correctly redirected to its origin
(usually and IdP endpoint).

Revision a8cc9cc6 (diff)
Added by Benjamin Dauvergne 3 months ago

retry HTTP requests 3 times (fixes #21783)

Retry is applied to access token request and user info requests (through
OAuth2Session). There is a small exponential backoff of 0.5 and 1s.

Also decrease log level of message for failure of retrieval of the
access token or the user info to the level WARNING, that's never been a
problem for the user, as he was correctly redirected to its origin
(usually and IdP endpoint).

History

#1 Updated by Benjamin Dauvergne over 1 year ago

J'ai aussi couvert la récupération des user_info, au passage je corrige un
warning de pytest à propos de pytest-catchlog.

#2 Updated by Benjamin Dauvergne over 1 year ago

En fait on jamais vraiment eu de souci ici, l'utilisateur pouvait déjà réessayer tranquillement de s'authentifier.

#3 Updated by Benjamin Dauvergne over 1 year ago

  • Status changed from Nouveau to En cours

#4 Updated by Josué Kouka over 1 year ago

Ack

#5 Updated by Benjamin Dauvergne over 1 year ago

  • Status changed from En cours to Résolu (à déployer)
  • % Done changed from 0 to 100

#6 Updated by Benjamin Dauvergne over 1 year ago

  • Target version set to 0.10

#7 Updated by Benjamin Dauvergne over 1 year ago

  • Status changed from Résolu (à déployer) to Fermé

Also available in: Atom PDF