Skip to content

fix sending mails from review apps : introduce HOST_QUALIFIED env var

Vincent Agnano requested to merge refactor/bundle-delayed-job into master

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_QUALIFIEDpartout

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

Merge request reports