[#14414] Horde_Mail factory ignores imp backend configuration
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@mausz.at
Created 2016-06-30 (1983 days ago)
Due
Updated 2016-11-19 (1841 days ago)
Assigned 2016-11-19 (1841 days ago)
Resolved
Milestone
Patch Yes

Comments
manuel@mausz.at 2016-06-30 12:29:04
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.

manuel@mausz.at 2016-06-30 22:36:31
Updated patch fixes mail sending from cli

Michael Rubinsky <mrubinsk@horde.org> 2016-11-19 16:36:34
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.

manuel-horde@mausz.at 2016-11-19 17:37:49
> 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.