Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Éviter le timeout en cas d’erreur de connexion superadmin #1764

Closed
n-b opened this issue Sep 29, 2021 · 0 comments · Fixed by #1769
Closed

Éviter le timeout en cas d’erreur de connexion superadmin #1764

n-b opened this issue Sep 29, 2021 · 0 comments · Fixed by #1769
Assignees
Labels

Comments

@n-b
Copy link
Contributor

n-b commented Sep 29, 2021

Actuellement

Si un superadmin tente d’accéder à la conseole superadmin sans être connecté, c’est à dire sans être passé par la page de connexion, l’app retourne une erreur, mais essaye aussi d’envoyer une erreur à sentry. Pour une raison inconnue, cette remontée d’erreur ne fonctionne pas, mais bloque le thread web pendant 60s.

Attendu

Idéalement, il faudrait simplement faire une redirection vers connexion_super_admin; je ne sais pas vraiment pourquoi on remonte une erreur.

Technique

Peut-être que c’est un sujet lié: il y a des choses bizarres sur l’accès à superadmin sur recette, et sur l’accès à superadmin/delayed_jobs sur les review apps. Il y a de quoi investiguer; mais en tout cas, il faut éviter de bloquer le thread 🤷

Côté sentry, c’est RDV-SOLIDARITES-GH.

NameError: undefined local variable or method `super_admin_github_omniauth_authorize_path'

@n-b n-b self-assigned this Sep 29, 2021
@n-b n-b added this to Backlog in Déploiement en production via automation Sep 29, 2021
@n-b n-b moved this from Backlog to En cours in Déploiement en production Sep 29, 2021
@n-b n-b mentioned this issue Sep 30, 2021
2 tasks
n-b added a commit that referenced this issue Oct 4, 2021
fixes #1764

I suppose this was broken since #975
@yaf yaf closed this as completed in #1769 Oct 4, 2021
Déploiement en production automation moved this from En cours to En recette Oct 4, 2021
@yaf yaf added the vu en demo label Oct 5, 2021
@yaf yaf moved this from En recette to En production in Déploiement en production Oct 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants