SugarDockerized icon indicating copy to clipboard operation
SugarDockerized copied to clipboard

Proposal. Change testsmtp to MailHog.

Open opxwebdev opened this issue 3 years ago • 3 comments

Hello Enrico,

For convenience, I suggest changing testsmtp to MailHog.

    mailhog:
        container_name: "sugar-mailhog"
        image: mailhog/mailhog:latest
        ports:
            - "8025:8025"

MailHog performs the same task as testsmtp, but it also has a simple interface for viewing emails.

I can make a pull request with configuration and documentation changes.

opxwebdev avatar Aug 21 '20 07:08 opxwebdev

Sounds good to me! Make sure you grep anywhere for sugar-smtp and fix up everything referring to it

Thanks for the idea and contribution!

esimonetti avatar Oct 18 '20 04:10 esimonetti

Hello Enrico,

I made a pull request with MailHog container. I also added a description of MailHog container to the README.md file.

It is also possible to configure the saving of emails to the file system. But I don't know if this is a good idea. What do you think about it?

opxwebdev avatar Nov 03 '20 23:11 opxwebdev

Hey there,

Would you be able to re-do the PR now that I merged a bunch of stuff? Any chance we can leave the host as sugar-smtp and the container name as well to provide as much as possible backward compatibility?

Thanks!

esimonetti avatar Jun 28 '21 09:06 esimonetti