Skip to content

allow to use a custom SMTP configuration

Ghost User requested to merge (removed):features/allow-smtp-configuration into master

Ce merge request permet d'utiliser une configuration personnalisée pour l'envoi des mails. Par défaut, on reste sur la configuration par défaut (localhost:25)

Il y a juste un point d'interrogation : pour préserver la compatibilité lors d'une mise à jour, on pourrait lors de l'appel au service de mail, vérifier que la variable de configuration existe (car c'est une nouvelle variable de configuration). Mais selon moi, cela ajouterait une dette technique pour plus tard, c'est pourquoi je ne l'ai pas mis en place :

// Actuellement avec le merge request
$mailService = new MailService($config['use_smtp'], $config['smtp_options']);

// Modification possible
$mailService = new MailService($config['use_smtp'], isset($config['smtp_options']) ? $config['smtp_options'] : []);

Merge request reports