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

Allow ignoring SMTP Host SSL errors #2380

Closed
ToasterUwU opened this issue Jul 11, 2024 · 1 comment
Closed

Allow ignoring SMTP Host SSL errors #2380

ToasterUwU opened this issue Jul 11, 2024 · 1 comment
Labels
feature User-facing features and product enhancements good first issue Good for newcomers

Comments

@ToasterUwU
Copy link
Contributor

Is your feature request related to a problem? Please describe.
I just tried the SMTP integration for reset password emails. Really cool idea, but if my SMTP server is in the local net and i dont have DNS setup in a specific way to where an actual domain will point to a local net address, i will not be able to use this.
When i tried to send me a reset password email for testing this feature, i got a error page, and in the log it says:

F, [2024-07-11T13:17:34.500187 #229] FATAL -- : [93efcad3-02b4-40f7-9fb4-966f0407926a]   
[93efcad3-02b4-40f7-9fb4-966f0407926a] OpenSSL::SSL::SSLError (SSL_connect returned=1 errno=0 peeraddr=192.168.178.3:25 state=error: certificate verify failed (hostname mismatch)):
[93efcad3-02b4-40f7-9fb4-966f0407926a]   
[93efcad3-02b4-40f7-9fb4-966f0407926a] app/controllers/users/passwords_controller.rb:21:in `create'
[93efcad3-02b4-40f7-9fb4-966f0407926a] app/controllers/application_controller.rb:72:in `switch_locale'

Describe the solution you'd like
I would like another environment variable in the SMTP section that is something like "SMTP_IGNORE_SSL_ERRORS", so that i can work around this issue in a clean manner.

Describe alternatives you've considered
Alternatively this could also be a "SMTP_VERIFY_SSL" where it needs to be actively set to "disable", but thats a thing you need to decide.

@ToasterUwU ToasterUwU added the feature User-facing features and product enhancements label Jul 11, 2024
@Floppy
Copy link
Collaborator

Floppy commented Jul 11, 2024

Pretty sure there's an ActionMailer setting for this, I'll see if I can expose it 👍🏻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature User-facing features and product enhancements good first issue Good for newcomers
Projects
Archived in project
Development

No branches or pull requests

2 participants