SMTP support - Githubissues

ualibraries / Guide-on-the-Side

The University of Arizona Libraries will no longer provide support for Guide on the Side. The code will remain openly available; however, UAL can no longer provide code fixes or upgrades.

https://ualibraries.github.io/Guide-on-the-Side/about.html

Other

66 stars 48 forks source link

SMTP support #106

Closed michaelhagedon closed 9 years ago

michaelhagedon commented 9 years ago

Because not everyone has easy access to get PHP mail() set up correctly.

This has been tested against Gmail; other testing would be good.

simpsonw commented 9 years ago

This looks good to me so far. I have tested it on Gmail under the following circumstances:

TLS (port 587) with valid credentials
SSL (port 465) with valid credentials
SSL (port 465) with invalid credentials (failed as expected with an error message)

I'd say we're pretty much ready to merge it, but I'd like to test it with another service first. Not surprisingly, the testing with Gmail's SMTP allowed me to send emails to Gmail addresses re: issue #72 .

simpsonw commented 9 years ago

I tried this with my personal U of A email credentials and it worked for me sending to Gmail and Mailinator. I'm all for merging it, unless @michaelhagedon objects.

michaelhagedon commented 9 years ago

No objections here. :-)