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

6.0.0-git

2019-06-24

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 (1089 days ago)
Due
Updated	2016-11-19 (947 days ago)
Assigned	2016-11-19 (947 days ago)
Resolved
Milestone
Patch	Yes

2016-11-19 17:37:49	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.

2016-11-19 16:36:34	Michael Rubinsky	Comment #3 Priority ⇒ 1. Low State ⇒ Feedback	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.

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

2016-06-30 12:29:04	manuel (at) mausz (dot) at	Comment #1 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	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.