From 51159117c9a6f88c189346dd14b4ed31fe33c5fb Mon Sep 17 00:00:00 2001 From: Pierre de La Morinerie Date: Tue, 15 Jun 2021 17:02:59 +0200 Subject: [PATCH] routes: replace the email param in the URL to a query param This basically moves from: /lien-envoye/foo@bar.com to: /lien-envoye?email=foo@bar.com This allow to properly censor the data in Matomo (so that email addresses don't end up stored in Matomo). --- config/routes.rb | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/config/routes.rb b/config/routes.rb index 9dba2d095..f5b4794a9 100644 --- a/config/routes.rb +++ b/config/routes.rb @@ -105,7 +105,8 @@ Rails.application.routes.draw do devise_scope :user do get '/users/no_procedure' => 'users/sessions#no_procedure' get 'connexion-par-jeton/:id' => 'users/sessions#sign_in_by_link', as: 'sign_in_by_link' - get 'lien-envoye/:email' => 'users/sessions#link_sent', constraints: { email: /.*/ }, as: 'link_sent' + get 'lien-envoye' => 'users/sessions#link_sent', as: 'link_sent' + get 'lien-envoye/:email' => 'users/sessions#link_sent', constraints: { email: /.*/ }, as: 'link_sent_legacy' # legacy, can be removed as soon as the previous line is deployed to production servers get '/users/password/reset-link-sent' => 'users/passwords#reset_link_sent' end