[#SAK-12703] message not sent if any user's address is wrong; tells user it was sent

Sakai

message not sent if any user's address is wrong; tells user it was sent

Created: 14-Jan-2008 14:06 Updated: 27-Oct-2008 10:57

Component/s:

Mailtool

Affects Version/s:

2.3.0, 2.4.0, 2.5.0, 2.3.1, 2.3.2, 2.4.1

File Attachments:

SAK-12703-validate-emails.patch (8 kb)

Image Attachments:

1. email_invalid.png
(86 kb)

Description

« Hide

in a site where one user has a bad address, mail tool send will fail, because sendPartial is not enabled. Not only does no one get the messaeg, but the user gets a confirmation screen saying it has been sent. This is two bugs;

1) send partial needs to be set
2) errors need to be returned to the user

I consider (2) to be a showstopper. I don't think people should be using a tool that claims to have sent mail when it actually didn't.

All

Comments

Change History

Subversion Commits

Sort Order:

Ascending order - Click to sort in descending order

[ Permlink | « Hide ]

SOO IL KIM - 15-Jan-2008 06:56

Charles,

Could you let me know which tag you are using? And please provide me what kind of bad email address you have tested. Thanks.

[ Permlink | « Hide ]

SOO IL KIM - 15-Jan-2008 07:26

Basically, Mailtool's version numbering is not associated with Sakai tags. Current trunk should have the most stable and bug-fixed version of Mailtool. As I remember, we have added many lines for correcting/enhancing email address validation of Mailtool in ~~SAK-11046~~.

So please try to use the current trunk and let me know if this issue still don't go away. Thanks.

[ Permlink | « Hide ]

Charles Hedrick - 17-Jan-2008 13:10

I used an address that is valid syntax and points to a valid mail server, but the address doesn't exist on the server. Unless you set sendpartial, JavaMail will abort the entire send.

See SAK-12750. That has a fix for both 2.5 and current trunk.

Trunk improves things, in that it detedts errors and reports them. But it doesn't set sendpartial, so a single bad address will abort the send.

[ Permlink | « Hide ]

SOO IL KIM - 18-Jan-2008 07:48

Applied patches in SAK-12750

[ Permlink | « Hide ]

Megan May - 05-Feb-2008 14:53

Updating 2.5.x to 2.5.0.rc1 since the tag has been cut

[ Permlink | « Hide ]

Renuka Devi Kaipa - 07-Feb-2008 07:22

I tested this on qa4-us. When I sent this email to invalid other recipient, I received error message, eventhough the message indicate that it failed. When I log in as student, student can see the message on site. Error message should be in red, it will be good if it differentiate the failed and successful mails.

[ Permlink | « Hide ]

Matthew Buckett - 27-Oct-2008 10:57

Attached patch that validates emails.