Produits Entr'ouvert » Corbo

• utilise le backend de mail de Django
• supporte inlining des CSS et des images
• signature DKIM

Il faudrait pouvoir simplement définir des templates HTML Django qui utilisent des feuilles de style Django ainsi que images avec des URLs absolues et/ou des images venant des statiques.

Pour chaque URL d'image il faut remplacer l'URL relative vers /static/img/xxx.png par une image attachée ou un lien absolu vers le site corbo (il faut récupérer le hostname depuis le tenant en cours).

[...] par une image attachée

Oui, parce que des images extérieures ça va être bloqué par les MUA. (pour la réalisation, cf http://code.activestate.com/recipes/473810/).

Je ne connais pas beaucoup de MUA qui bloquent le contenu web et je pensais plutôt envoyer des mails plus legers, sans les statics inclus.

Serghei Mihai a écrit :

Je ne connais pas beaucoup de MUA qui bloquent le contenu web

Aujourd'hui : tous.

De toute façon la lib pointée gère tout ça, la discussion n'a pas lieu d'être, envoyons donc des fichiers attachés.

Pour les images, les fichiers "media/" de manière générale, il y a une API dans Django (https://docs.djangoproject.com/en/dev/ref/files/storage/), c'est ça qu'il faut utiliser, parce qu'on l'a déjà adapté pour gérer le multitenant.

announce "%s" delivered to %s destinations

C'est un peu exagéré, "delivered", on n'en sait rien, on sait juste que le SMTP local a avalé le mail.

if result.status_code == 250:

Si ce n'est pas le cas c'est que le serveur SMTP local a un gros soucis, ce serait quand même important de le logguer très fort. Et d'enregistrer les destinataires pour qui ça n'a pas fonctionné (ce qui pourra être utilisé, plus tard, pour réessayer).

mail_from=settings.CORBO_DEFAULT_FROM_EMAIL)

J'utiliserais le paramétrage qui existe déjà (DEFAULT_FROM_EMAIL).

Au niveau des logs j'irai franco, une ligne par destinataire et par campagne ça ne me parait pas exagéré (niveau INFO), c'est un peu le but de l'outil.

Il faudrait un test qui vérifie que la CSS et les images sont bien inlinés.

Il ne manque pas une commande de management pour appeler l'envoi ?

Yep, à cause d'un patch j'ai proposé en upstream qui n'a pas encore été releasé sur pypi: https://github.com/smihai/python-emails/commit/6a2e6264f4048bd001b22b95f51980d6ac908d4f
Je modifie le setup pour récuperer la version de notre depot.

Et concernant :

Il ne manque pas une commande de management pour appeler l'envoi ?

Comme j'essayais de dire, il manque la commande de management (dont j'ai maintenant le nom, send_announces).

Au passage j'ai poussé le patch pour fixer les tests.