souramoo / commentoplusplus

Commento with out of the box patches and updates to add useful features and fixes. Also with one-click deploy to Heroku so you can get up and running fast.
MIT License
389 stars 62 forks source link

add STARTTLS email encryption option #134

Closed Loosetooth closed 1 year ago

Loosetooth commented 1 year ago

As mentioned in #133, STARTTLS did not seem to work for me.

This pull request uses the USE_STARTTLS environment variable to enable STARTTLS encryption or not.

To get it working for me, I also had to add the ServerName property in the server.TLSConfig variable. If I understand correctly, it shouldn't be a problem to always add the ServerName property.

I guess this should be tested with some other email setups before merging.