6.0.0-git
2019-03-20

[#154] Presentation of sendmail error codes
Summary Presentation of sendmail error codes
Queue IMP
Queue Version HEAD
Type Enhancement
State Resolved
Priority 1. Low
Owners
Requester xidus (at) xidus (dot) net
Created 2004-05-04 (5433 days ago)
Due
Updated 2005-06-16 (5025 days ago)
Assigned 2004-05-17 (5420 days ago)
Resolved 2005-06-16 (5025 days ago)
Milestone
Patch No

History
2005-06-16 02:30:29 Chuck Hagenbuch State ⇒ Resolved
 
2005-06-15 20:54:55 Matt Selsky Comment #16 Reply to this comment
Committed.
2005-06-15 16:40:43 Jan Schneider Comment #15 Reply to this comment
Much better, go ahead.
2005-06-15 16:06:57 Matt Selsky Comment #14
New Attachment: Compose[1].patch Download
Reply to this comment
How's this?
2005-06-15 07:58:42 Jan Schneider Comment #13 Reply to this comment
Please put the gettext strings in double quotes, and make only the 
varying error messages translateable.
2005-06-15 05:06:03 Matt Selsky Comment #12
New Attachment: Compose.patch Download
Reply to this comment
This needs more testing, but I think we're on the right track.
2005-06-15 03:37:40 Chuck Hagenbuch Comment #11 Reply to this comment
Here's the Mail_sendmail patch...
Committed, thanks.
2005-06-15 03:21:14 Matt Selsky Comment #10
New Attachment: sendmail.php.patch Download
Reply to this comment
Here's the Mail_sendmail patch...
2005-06-15 03:05:45 Chuck Hagenbuch Comment #9 Reply to this comment
I think the ideal solution would be to modify the PEAR::raiseError() 
call in Mail_sendmail to include the error code as a seperate 
parameter as well (it's in the raiseError() definition). Then in IMP, 
if there's an error code set, you can introduce the translatable 
strings. We shouldn't rely on the Mail_sendmail string staying the 
same, but the error code should be reliable.



Only downside is it requires changes to both packages, but it degrades 
to the current behavior just fine either way.
2005-06-14 19:43:06 Matt Selsky Comment #8 Reply to this comment
Is it safe to rely on Mail_sendmail to never change the strings it 
returns?  If it is, I can move this code to imp/lib/Compose.php and 
only attempt to parse the error message if $conf['mailer']['type'] == 
'sendmail'.
2005-06-14 08:30:11 Jan Schneider Comment #7 Reply to this comment
If we do it in IMP we could use gettext and translate the strings.
2005-06-14 02:38:06 Matt Selsky Comment #6
State ⇒ Feedback
New Attachment: sendmail.patch Download
Reply to this comment
This patch lets the PEAR module Mail_sendmail know about the sendmail 
error codes and what they mean.  Or should the logic be more inside of 
IMP?
2005-02-18 22:16:26 Jan Schneider Taken from Horde DevelopersHorde Developers
 
2005-02-18 21:04:06 Jan Schneider Comment #4
State ⇒ Accepted
Version ⇒ HEAD
Reply to this comment
New features go to HEAD only.
2004-05-17 15:34:52 Jan Schneider Priority ⇒ 1. Low
Assigned to Horde DevelopersHorde Developers
 
2004-05-17 15:34:51 Jan Schneider State ⇒ Assigned
 
2004-05-04 18:00:05 Chuck Hagenbuch Comment #3
Type ⇒ Enhancement
Reply to this comment
Patch?
2004-05-04 17:58:53 xidus (at) xidus (dot) net Comment #2 Reply to this comment
Even worse, in the case of EX_NOUSER, the user is given the impression 
that the mail was sent to NONE of the intended recipients.  However, 
if the mail was addressed to multiple recipients, it has been 
successfully sent to the extant recipients.  This easily leads to 
users clicking "send" multiple times to try to clear the error, 
sending multiple copies to the recipients that DO exist.



Handling in this case, with multiple recipients and one or more 
unknown to local sendmail, is very poor.  IMP should fail to send the 
message completely AND report the unknown addresses (reported by 
sendmail on stderr).
2004-05-04 17:51:57 xidus (at) xidus (dot) net Comment #1
Type ⇒ Bug
State ⇒ Unconfirmed
Priority ⇒ 2. Medium
Queue ⇒ IMP
Reply to this comment
When using sendmail locally to inject e-mail, return codes from 
sendmail are displayed very cryptically.  IE: "There was an error 
sending your message: sendmail returned error code 67"  Sendmail only 
returns 6-7 error codes, shown in sendmail(8), and defined in 
/usr/include/sysexits.h.  Readable messages for these would be useful. 
  For instance, 67 is EX_NOUSER, which means one of the addressees 
didn't exist.  The user sending mail would much rather see that than 
"error 67"



Thanks...

Saved Queries