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

ActiveRecord::RecordNotFound: Couldn't find Rdv with 'id'=3508010 [WHERE ("rdvs"."id" IN (SELECT "rdvs"."id" FROM "rdvs" INNER ... #4636

Open
sentry-sentry-incubateur-net bot opened this issue Sep 16, 2024 · 1 comment

Comments

@sentry-sentry-incubateur-net
Copy link

Il y a un nombre non négligeable de 404 sur les pages users/rdvs#show (+ de 100 par mois)

Une bonne partie de ces erreurs provient de la page de vérification des initiales (visible via le referer dans Sentry Discover)

Il s’agit probablement de rendez-vous qui ont été supprimés par un agent entre le moment où ils ont été pris depuis RDVI avec le SMS envoyé et le moment où l’usager accède au lien.

C’est une mauvaise expérience utilisateur, l’usager voit "Page introuvable" sans plus d’infos.

On pourrait par exemple toujours afficher un message plus spécifique sur cette 404 : « le RDV auquel vous essayez d’accéder n’existe pas ou n’existe plus. Un agent l’a probablement annulé » ou qqch comme ça

On pourrait aussi probablement ignorer ces 404 et ne pas les lever à Sentry du tout, c’est un cas normal.

Sentry Issue: LAPINS-1D4

ActiveRecord::RecordNotFound: Couldn't find Rdv with 'id'=3508010 [WHERE ("rdvs"."id" IN (SELECT "rdvs"."id" FROM "rdvs" INNER JOIN "participations" ON "participations"."rdv_id" = "rdvs"."id" INNER JOIN "users" ON "users"."id" = "participations"."user_id" INNER JOIN "motifs" ON "motifs"."id" = "rdvs"."motif_id" WHERE ("users"."id" = $1 OR "users"."deleted_at" IS NULL AND "users"."responsible_id" = $2) AND "motifs"."visibility_type" IN ($3, $4)) OR "rdvs"."id" IN (SELECT "rdvs"."id" FROM "rdvs" INNER JOIN "motifs" ON "motifs"."id" = "rdvs"."motif_id" WHERE "motifs"."collectif" = $5 AND "motifs"."bookable_by" IN ($6, $7)))] (ActiveRecord::RecordNotFound)
  app/controllers/users/rdvs_controller.rb:112:in `set_rdv'
    @rdv = policy_scope(Rdv).find(params[:id])
  config/initializers/cookie_overflow_notify.rb:10:in `call'
    @app.call(env)
  config/initializers/fallback_error_middleware.rb:8:in `call'
    response = @app.call(env)
...
(245 additional frame(s) were not displayed)
@francois-ferrandis
Copy link
Contributor

francois-ferrandis commented Sep 16, 2024

Il s’agit probablement de rendez-vous qui ont été supprimés par un agent entre le moment où ils ont été pris depuis RDVI avec le SMS envoyé et le moment où l’usager accède au lien.

@adipasquale Je suis retombé sur une issue qui date d'avirl 2023 où j'identifie une autre cause pour ces 404 : les RDVs invisibles. #3473

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 🔖 Ready
Development

No branches or pull requests

1 participant