https://dev.entrouvert.org/https://dev.entrouvert.org/favicon.ico?15861920342021-02-23T15:29:54ZRedmine Entr’ouvertAuthentic 2 - Development #51374: misc: absence de {% autoescape %} dans certains templates de mailhttps://dev.entrouvert.org/issues/51374?journal_id=2894342021-02-23T15:29:54ZBenjamin Dauvergne
<ul><li><strong>Fichier</strong> <a href="/attachments/52176">0001-misc-use-non-autoescaping-template-render-for-email-.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/52176/0001-misc-use-non-autoescaping-template-render-for-email-.patch">0001-misc-use-non-autoescaping-template-render-for-email-.patch</a> ajouté</li><li><strong>Statut</strong> changé de <i>Nouveau</i> à <i>Solution proposée</i></li><li><strong>Patch proposed</strong> changé de <i>Non</i> à <i>Oui</i></li></ul> Authentic 2 - Development #51374: misc: absence de {% autoescape %} dans certains templates de mailhttps://dev.entrouvert.org/issues/51374?journal_id=2894472021-02-23T15:57:50ZBenjamin Dauvergne
<ul><li><strong>Dupliqué par</strong> <i><a class="issue tracker-1 status-6 priority-4 priority-default closed" href="/issues/51376">Bug #51376</a>: build jenkins en échec</i> ajouté</li></ul> Authentic 2 - Development #51374: misc: absence de {% autoescape %} dans certains templates de mailhttps://dev.entrouvert.org/issues/51374?journal_id=2894492021-02-23T15:59:06ZBenjamin Dauvergne
<ul><li><strong>Fichier</strong> <a href="/attachments/52180">0001-misc-use-non-autoescaping-template-render-for-email-.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/52180/0001-misc-use-non-autoescaping-template-render-for-email-.patch">0001-misc-use-non-autoescaping-template-render-for-email-.patch</a> ajouté</li></ul> Authentic 2 - Development #51374: misc: absence de {% autoescape %} dans certains templates de mailhttps://dev.entrouvert.org/issues/51374?journal_id=2894682021-02-23T16:49:32ZValentin Deniaud
<ul><li><strong>Statut</strong> changé de <i>Solution proposée</i> à <i>En cours</i></li></ul><p>(toujours rouge)</p> Authentic 2 - Development #51374: misc: absence de {% autoescape %} dans certains templates de mailhttps://dev.entrouvert.org/issues/51374?journal_id=2894852021-02-23T18:15:02ZBenjamin Dauvergne
<ul><li><strong>Fichier</strong> <a href="/attachments/52188">0001-misc-use-non-autoescaping-template-render-for-email-.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/52188/0001-misc-use-non-autoescaping-template-render-for-email-.patch">0001-misc-use-non-autoescaping-template-render-for-email-.patch</a> ajouté</li><li><strong>Statut</strong> changé de <i>En cours</i> à <i>Solution proposée</i></li></ul><p>Sans modifier la configuration globale c'est moins fatiguant.</p> Authentic 2 - Development #51374: misc: absence de {% autoescape %} dans certains templates de mailhttps://dev.entrouvert.org/issues/51374?journal_id=2895242021-02-24T08:25:39ZValentin Deniaud
<ul></ul><p>Pas fan du tout de ce <code>render_plain_text_template_to_string</code>, je préférais le premier patch. Sinon sans toucher aux backends cette approche semble fonctionner :</p>
<pre>
@@ -664,2 +665,3 @@ def send_templated_mail(user_or_email, template_names, context=None, with_html=T
ctx.update(context)
+ plain_text_ctx = make_context(ctx, request=request, autoescape=False)
@@ -667,3 +669,4 @@ def send_templated_mail(user_or_email, template_names, context=None, with_html=T
subject_template_names += legacy_subject_templates or []
- subject = render_plain_text_template_to_string(subject_template_names, ctx, request=request).strip()
+ template = select_template(subject_template_names)
+ subject = template.template.render(plain_text_ctx).strip()
@@ -671,3 +674,4 @@ def send_templated_mail(user_or_email, template_names, context=None, with_html=T
body_template_names += legacy_body_templates or []
- body = render_plain_text_template_to_string(body_template_names, ctx, request=request)
+ template = select_template(body_template_names)
+ body = template.template.render(plain_text_ctx)
</pre> Authentic 2 - Development #51374: misc: absence de {% autoescape %} dans certains templates de mailhttps://dev.entrouvert.org/issues/51374?journal_id=2897432021-02-24T21:57:46ZBenjamin Dauvergne
<ul><li><strong>Fichier</strong> <a href="/attachments/52219">0001-misc-use-non-autoescaping-template-render-for-email-.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/52219/0001-misc-use-non-autoescaping-template-render-for-email-.patch">0001-misc-use-non-autoescaping-template-render-for-email-.patch</a> ajouté</li></ul><p>adopté.</p> Authentic 2 - Development #51374: misc: absence de {% autoescape %} dans certains templates de mailhttps://dev.entrouvert.org/issues/51374?journal_id=2897752021-02-25T08:01:32ZValentin Deniaud
<ul><li><strong>Statut</strong> changé de <i>Solution proposée</i> à <i>Solution validée</i></li></ul> Authentic 2 - Development #51374: misc: absence de {% autoescape %} dans certains templates de mailhttps://dev.entrouvert.org/issues/51374?journal_id=2898432021-02-25T09:34:32ZBenjamin Dauvergne
<ul><li><strong>Statut</strong> changé de <i>Solution validée</i> à <i>Résolu (à déployer)</i></li></ul><pre>commit 562cb43476921eb071ad67161d9461b7d09e9c3c
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Tue Feb 23 16:25:37 2021 +0100
misc: use non-autoescaping template render for email text bodies and subjects (#51374)</pre> Authentic 2 - Development #51374: misc: absence de {% autoescape %} dans certains templates de mailhttps://dev.entrouvert.org/issues/51374?journal_id=2911212021-03-04T09:16:10ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Statut</strong> changé de <i>Résolu (à déployer)</i> à <i>Solution déployée</i></li></ul>