Améliorer les erreurs d’authentification Oauth remontées à Sentry

[adipasquale]

Contexte

ces erreurs sur Sentry sont difficiles à investiguer :

Omniauth for Franceconnect failed on www.rdv-solidarites.fr: undefined method `to_sym' for nil

Solution proposée

Remplacer le capture_message par un capture_exception qui devrait nous permettre d’avoir la stack trace de l’erreur et comprendre de quel to_sym on parle.

J’ai réussi à tester en local en :

• définissant les credentials SENTRY_DSN et FRANCECONNECT dans mon .env
• supprimant le wrapper devise_scope :user dans routes.rb autour du callback FC

on a la stacktrace et on peut voir d’où vient ce nil.to_sym (teaser : il vient de omniauth_openid_connect on dirait)

TODO

• [x] Tester en local une fois les credentials France Connect dev récupérés
• [x] valider avec @francois-ferrandis qu’il n’y avait pas une raison précise de faire capture_message, peut-être pour grouper les erreurs. J’ai creusé les git blames mais je n’ai jamais vu de capture_exception

[adipasquale]

je me demande si le set_context fonctionne event sentry avant avec les breadcrumbs event sentry après cette PR

[francois-ferrandis]

je me demande si le set_context fonctionne event sentry avant avec les breadcrumbs event sentry après cette PR

Je me demande si c'est pas une question de scrubbing, parce que je vois que le breadcrumb était nommé Omniauth env values et ressortait [Filtered]. Donc si le nouveau context s'appelle omniauth_env, peut-être qu'il est ignoré ?