Summary | Horde_Mail factory ignores imp backend configuration |
Queue | IMP |
Queue Version | 6.2.14 |
Type | Bug |
State | Feedback |
Priority | 1. Low |
Owners | |
Requester | manuel (at) mausz (dot) at |
Created | 2016-06-30 (1706 days ago) |
Due | |
Updated | 2016-11-19 (1564 days ago) |
Assigned | 2016-11-19 (1564 days ago) |
Resolved | |
Milestone | |
Patch | Yes |
context? From what I can tell, it's only used from Core in response
to processing the prefs form via services/prefs.
adds a foreign identity/email address.
Priority ⇒ 1. Low
State ⇒ Feedback
context? From what I can tell, it's only used from Core in response to
processing the prefs form via services/prefs.
New Attachment: horde_mail[1].patch
Type ⇒ Bug
State ⇒ Unconfirmed
Priority ⇒ 2. Medium
Summary ⇒ Horde_Mail factory ignores imp backend configuration
Queue ⇒ IMP
Milestone ⇒
Patch ⇒ Yes
New Attachment: horde_mail.patch
the configuration from imp/config/backends gets completely ignored. If
the global mailer type is set to 'smtp' Horde_Mail overwrites username
+ password from users credentials, however all other parameters are
left in place (most importantly host) which might result in invalid
smtp auth credentials.
The attached patch simply overwrites the Horde_Mail injector interface
with IMP_Factory_Mail. The latter already passes the right credentials
to it's base class.
There's also another bug I wasn't able to track down: Horde_Mail tries
to send the mail in a loop if smtp auth fails.