6.0.0-git
2018-12-16

[#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 (at) mausz (dot) at
Created 2016-06-30 (899 days ago)
Due
Updated 2016-11-19 (757 days ago)
Assigned 2016-11-19 (757 days ago)
Resolved
Milestone
Patch Yes

History
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 Download
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 Download
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.

Saved Queries