fix sending mails from review apps : introduce HOST_QUALIFIED env var
Created by: adipasquale
actuellement :
- en demo :
HOST=demo.rdv-solidarites.fr
- sur les review apps:
HOST=https://demo-rdv-solidarites-pr653.osc-fr1.scalingo.io
-> le https est 'en trop' sur les review apps. Cela cause des problemes, dont l'envoi de mails.
On ne peut pas corriger en enlevant le HTTPS du HOST des review apps, car il est mis automatiquement lors de la création des review apps, on n'a pas la main dessus.
On ne peut donc pas faire de fix sûr en une seule PR je pense, car il faudrait déployer le code + changer simultanément la variable d'env HOST
.
Solution proposée :
- introduction d'une nouvelle variable d'env
HOST_QUALIFIED
qui contient le HOST avec le protocole - on s'assure qu'elle est bien configurée manuellement sur tous les envs actuels (demo, prod, review apps)
- on deploie cette PR
- une fois que demo et prod sont déployés :
- on modifie
HOST
en demo et en prod pour rajouter le protocole - on fait une nouvelle PR pour revenir a l'utilisation de
HOST
tout court, on supprime completementHOST_QUALIFIED
partout
- on modifie
unrelated : Le fait d'utiliser bundle
pour lancer delayed_job semble réparer le fait qu'on n'ait pas les exceptions dans sentry depuis les jobs