Projet

Général

Profil

Bug #3692

mauvais URLs dans les mails quand wcs est derrière un proxy

Ajouté par Thomas Noël il y a plus de 10 ans. Mis à jour il y a plus de 9 ans.

Statut:
Fermé
Priorité:
Immediat
Assigné à:
Thomas Noël
Version cible:
-
Début:
01 octobre 2013
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Planning:

Description

Configuration: wcs installé sur une machine "local" derrière un proxy "public.domaine.tld"

Dans ce cas, certains mails utilisent request.get_url(), donc http://local. Il faut leur faire utiliser publisher.get_frontoffice_url()

Vu sur :

  • mail de validation d'inscription
  • mail de demande de nouveau mot de passe

Fichiers

0001-makes-sure-frontoffice-url-is-used-in-all-relevant-p.patch (5,94 ko) 0001-makes-sure-frontoffice-url-is-used-in-all-relevant-p.patch Frédéric Péters, 02 janvier 2014 16:21

Révisions associées

Révision 7caac66b (diff)
Ajouté par Thomas Noël il y a plus de 10 ans

add token variable in mails (#3692)

Historique

#1

Mis à jour par Thomas Noël il y a plus de 10 ans

Dans ce mails, je propose en première approche d'envoyer le token, ce qui permettra déjà de "forger" des mails avec la bonne URL en modifiant le texte du courriel :

diff --git a/wcs/qommon/ident/password.ptl b/wcs/qommon/ident/password.ptl
index 5129252..bfa95af 100644
--- a/wcs/qommon/ident/password.ptl
+++ b/wcs/qommon/ident/password.ptl
@@ -347,6 +347,7 @@ class MethodDirectory(Directory):
         data = {
             'change_url': get_request().get_url() + '?t=%s&a=cfmpw' % token.id,
             'cancel_url': get_request().get_url() + '?t=%s&a=cxlpw' % token.id,
+            'token': '%s' % token.id,
             'time': misc.localstrftime(time.localtime(token.expiration)),
         }

@@ -670,6 +671,7 @@ $(function() {
             'email': user.email,
             'website': get_cfg('sitename'),
             'token_url': token_url,
+            'token': '%s' % token.id,
             'username': account.id,
             'password': password,
             'admin_email': passwords_cfg.get('admin_email', ''),
@@ -1398,7 +1400,7 @@ class AccountsDirectory(AccessControlled, Directory):

 EmailsDirectory.register('password-subscription-notification',
         N_('Subscription notification for password account'),
-        N_('Available variables: email, website, token_url, admin_email, username, password'),
+        N_('Available variables: email, website, token_url, token, admin_email, username, password'),
         category = N_('Identification'),
         default_subject = N_('Subscription Confirmation'),
         default_body = N_('''\
@@ -1418,7 +1420,7 @@ to [admin_email].

 EmailsDirectory.register('change-password-request',
         N_('Request for password change'),
-        N_('Available variables: change_url, cancel_url, time'),
+        N_('Available variables: change_url, cancel_url, token, time'),
         category = N_('Identification'),
         default_subject = N_('Change Password Request'),
         default_body = N_("""\
#3

Mis à jour par Frédéric Péters il y a plus de 10 ans

  • Statut changé de Nouveau à Résolu (à déployer)
commit 16e42f30b687663ccf2fda21c4ea5366274518d9
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Thu Jan 2 16:20:24 2014 +0100

    makes sure frontoffice url is used in all relevant places (#3692)
#4

Mis à jour par Thomas Noël il y a presque 10 ans

  • Statut changé de Résolu (à déployer) à Solution déployée
#5

Mis à jour par Thomas Noël il y a plus de 9 ans

  • Statut changé de Solution déployée à Fermé

Formats disponibles : Atom PDF