Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Message Delivery #53

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Conversation

hundredwatt
Copy link

This pull request adds a "Deliver" button to the web interface next to the "Download" button.

In the backend, there is now a DeliveryService class that is initialized with a message hash and responds to #deliver!. The current implementation uses a basic Net::SMTP setup for delivery. SMTP options are provided on the command line with defaults setup for GMail. So if you are using GMail, the minimum necessary configuration is:

mailcatcher --delivery-user-name [email protected] --delivery-password YOURPASS

There is also a '--delivery-recipient' option which overwrites the recipient field on all delivered messages. This allows you to specify an email address that you would like all messages to be delivered to.

"Deliver" button is replaced with text while waiting for delivery
Currently only supports SMTP. Configuration is provided at the command
line. Default values are setup for GMail delivery.
@hundredwatt
Copy link
Author

This is in response to #4

@sj26
Copy link
Owner

sj26 commented Oct 29, 2012

Great idea, not sure how comfortable I am with the gmail integration. I'd be more inclined to just act like a first-class SMTP server, however that raises problems via ISPs. Also want the deliveree's address to be changeable in the web interface.

@hundredwatt
Copy link
Author

The current setup can take in any SMTP configuration. I chose to add GMail as the default SMTP configuration figuring it was relatively common and easiest to setup.

For the deliveree's address, what use case do you have in mind for wanting to change their address via the interface?

@sepehr
Copy link

sepehr commented Jan 2, 2014

+1, great idea.

@sj26 sj26 force-pushed the master branch 2 times, most recently from aa7bc8e to 23398d7 Compare January 27, 2015 21:44
@bogdal
Copy link

bogdal commented Jan 7, 2016

It would be great to have this merged.

@jehon
Copy link

jehon commented Jul 19, 2016

Great idea. When will it be merged?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants