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

Various fixes for Mail Agent #8451

Open
wants to merge 2 commits into
base: release-3.0
Choose a base branch
from

Conversation

jdarwood007
Copy link
Member

Fixes #8447

@jdarwood007
Copy link
Member Author

@Oldiesmann Test and verify

@Oldiesmann
Copy link
Contributor

It works with these changes

@Oldiesmann
Copy link
Contributor

Update after some more testing, though I'm not sure if this is related...
Any email that goes through the mail queue just sits there forever, unless one of them happens to be going to whoever the test email is sent to.

On a test board on my server, I sent out two newsletters, with known working mail settings. The initial one sat there for well over an hour until I sent a test email, then both the newsletter and test email arrived at the same time. I tried again and the same thing happened - newsletter sat in the queue until I sent a test email. However, the emails for the regular account I created are still sitting there, and trying to forcefully clear the queue doesn't work

I got the same result on my local test board using PapercutSMTP - if I send a newsletter, it sits there forever. As soon as I send a test email, the newsletter gets sent right after

@Oldiesmann
Copy link
Contributor

Final update of the night. The two that got stuck in the queue finally got sent - thirty minutes after I sent the second one and nearly two and a half hours after the initial mail was sent. Again I'm not sure if it's a problem with the mail agent code or something else mail-related as I haven't tested newsletters prior to this and I don't think I still have an SMF3 test board from before this was implemented.

@jdarwood007
Copy link
Member Author

Any email sitting there in the queue means the queue is not being processed. I didn't touch that with this PR. I haven't looked into what 3.0 is doing for the queue. 2.1 we are adjusting the queue to handle emails not sending and clogging up the queue.

@Oldiesmann
Copy link
Contributor

Ok, then it wouldn't be a problem with this. Email works fine other than the mail queue issues.

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.

[3.0]: Unable to send test email with MailAgent
2 participants