Skip to content

setting tls to 465. was having issues sending email through campus ne… - #7

Merged
davezuckerman merged 1 commit into
mainfrom
dockerfile-update
Jan 16, 2026
Merged

setting tls to 465. was having issues sending email through campus ne…#7
davezuckerman merged 1 commit into
mainfrom
dockerfile-update

Conversation

@davezuckerman

@davezuckermandavezuckerman commented Jan 16, 2026

Copy link
Copy Markdown
Contributor

using tls on port 465. Also moved the Email confirmation so it's only triggered on successful email submission

Comment threadlib/mailer.rb Outdated
Comment threadlib/mailer.rb
…twork
removed starttls_auto and using tls: true instead, using logging.error as opposed to info for error

@danschmidt5189danschmidt5189 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm :shipit:

@davezuckerman
davezuckerman merged commit ebd7913 into mainJan 16, 2026
5 checks passed
@davezuckerman
davezuckerman deleted the dockerfile-update branch January 16, 2026 21:22
Sign up for freeto 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.

2 participants

@davezuckerman@danschmidt5189