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.

Sign up for GitHub

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

Jump to bottom

Timeouts not working for synchronous sendMail calls. #229

Closed
ghost opened this issue Oct 14, 2019 · 3 comments
Closed

Timeouts not working for synchronous sendMail calls. #229

ghost opened this issue Oct 14, 2019 · 3 comments
Assignees
@bbottema
Labels
bug Priority-Medium
Milestone
5.5.0

Comments

@ghost
Copy link

ghost commented Oct 14, 2019 •
edited by ghost
Loading

Is there a reason why for synchronous calls no timeouts are configured?
The workaround is to set the timeouts via properties.

MailSender.send(final Email email, final boolean async)
...
if (async)
...
configureSessionWithTimeout(...)

I'm using SimpleJavaMail:5.4.0
@ghost ghost changed the title Timeouts not working for synchronous sendEmail calls. Timeouts not working for synchronous sendMail calls. Oct 14, 2019
@bbottema
Copy link
Owner

If I remember correctly, this was a bug I encountered while implementing advanced batch support for 6.0.0. I'll dive into the 5.x.y code for you somewhere this week to confirm.

bbottema added a commit that referenced this issue Oct 15, 2019
@bbottema
#229 always apply connection properties
e5fc50b
@bbottema
Copy link
Owner

I've confirmed the bug and can inform you it's fixed in the upcoming 6.0.0. I'll have a minor update out soon for version 5.

@bbottema bbottema added this to the 5.5.0 milestone Oct 15, 2019
@bbottema bbottema self-assigned this Oct 15, 2019
@bbottema
Copy link
Owner

v5.5.0 released with fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bbottema bbottema

Labels
bug Priority-Medium
Projects
None yet
Milestone
5.5.0
Development

No branches or pull requests
1 participant
@bbottema