Tickets :: [#14414] Horde_Mail factory ignores imp backend configuration

6.0.0-beta1

10/24/25

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	06/30/2016 (3403 days ago)
Due
Updated	11/19/2016 (3261 days ago)
Assigned	11/19/2016 (3261 days ago)
Resolved
Github Issue Link
Github Pull Request
Milestone
Patch	Yes

11/19/2016 05:37:49 PM	manuel-horde (at) mausz (dot) at	Comment #4	Reply to this comment
Where do you see the verifyIdentity code being called from IMP's context? From what I can tell, it's only used from Core in response to processing the prefs form via services/prefs. It's been a while but as far as i recall this gets triggered if a user adds a foreign identity/email address.

11/19/2016 04:36:34 PM	Michael Rubinsky	Comment #3 State ⇒ Feedback Priority ⇒ 1. Low	Reply to this comment
Where do you see the verifyIdentity code being called from IMP's context? From what I can tell, it's only used from Core in response to processing the prefs form via services/prefs.

06/30/2016 10:36:31 PM	manuel (at) mausz (dot) at	Comment #2 New Attachment: horde_mail[1].patch	Reply to this comment
Updated patch fixes mail sending from cli

06/30/2016 12:29:04 PM	manuel (at) mausz (dot) at	Comment #1 Priority ⇒ 2. Medium New Attachment: horde_mail.patch Patch ⇒ Yes Milestone ⇒ Summary ⇒ Horde_Mail factory ignores imp backend configuration Type ⇒ Bug State ⇒ Unconfirmed Queue ⇒ IMP	Reply to this comment
If a mail gets send through Horde_Mail (e.g. identity verification) 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.