6.0.0-git
2019-12-14

[#14005] Inconsistent message for SMTP error 554
Summary Inconsistent message for SMTP error 554
Queue IMP
Queue Version Git master
Type Enhancement
State Assigned
Priority 1. Low
Owners Horde Developers (at) , mrubinsk (at) horde (dot) org
Requester ddolgopolov (at) odin (dot) com
Created 2015-06-05 (1653 days ago)
Due
Updated 2015-07-20 (1608 days ago)
Assigned 2015-07-20 (1608 days ago)
Resolved
Milestone
Patch No

History
2015-07-20 14:16:31 Michael Rubinsky Assigned to Michael Rubinsky
Assigned to Horde DevelopersHorde Developers
 
2015-07-20 14:16:17 Michael Rubinsky Comment #4
Type ⇒ Enhancement
Priority ⇒ 1. Low
State ⇒ Assigned
Reply to this comment
Accepting as an enhancement, for updating the language of the error 
displayed to be more general,  as suggested by MIchael in his comment.
2015-06-08 04:03:11 ddolgopolov (at) odin (dot) com Comment #3 Reply to this comment
Michael, it would be nice to show "There was an error when trying to 
send the message".
2015-06-05 15:55:06 Michael Slusarz Comment #2
State ⇒ Feedback
Reply to this comment
You are confusing the RFC requirement - which concerns error messages 
output by the server intended to be displayed by an *admin* - with the 
error message intended to be displayed to a *user*.

For 554 errors, without extended error codes, there is not a single, 
simple message that works for all server responses.  (If anything, we 
should probably be even MORE general with the response, e.g. "There 
was an error when trying to send the message").

What you are asking for is already provided.  Not only is the error 
message available in SMTP debug logs, but it will be logged to the 
Horde error log (at least as of IMP 7.0).

You absolutely cannot be appending the direct content of SMTP error 
messages to user displayable data.  That is a recipe for disaster.
2015-06-05 10:20:03 ddolgopolov (at) odin (dot) com Comment #1 (Private)
Type ⇒ Bug
State ⇒ Unconfirmed
Priority ⇒ 1. Low
Summary ⇒ Inconsistent message for SMTP error 554
Queue ⇒ IMP
Milestone ⇒
Patch ⇒ No
[Hidden]

Saved Queries