Skip to content

Conversation

@valerieqq
Copy link
Contributor

No description provided.

properties.put("mail.smtp.host", host);
properties.put("mail.smtp.port", "587");
properties.put("mail.smtp.auth", "true");
properties.put("mail.smtp.starttls.enable", "true");
Copy link
Owner

Choose a reason for hiding this comment

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

What was the use case for this?
Perhaps it makes sense to bind a property here and let the user change this value?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Without this property sendEmail method is not working:

SEVERE: 530-5.7.0 Must issue a STARTTLS command first. For more information, go to
530-5.7.0 https://support.google.com/a/answer/3221692 and review RFC 3207
530 5.7.0 specifications. a640c23a62f3a-ac3efd569fesm853582966b.173 - gsmtp

com.sun.mail.smtp.SMTPSendFailedException: 530-5.7.0 Must issue a STARTTLS command first. For more information, go to
530-5.7.0 https://support.google.com/a/answer/3221692 and review RFC 3207
530 5.7.0 specifications. a640c23a62f3a-ac3efd569fesm853582966b.173 - gsmtp

Copy link
Owner

Choose a reason for hiding this comment

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

Thats odd, it worked before 🤔

@Umutayb Umutayb merged commit 6ac5534 into Umutayb:main Mar 25, 2025
2 checks passed
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.

2 participants