[Tickets #12448] Re: IMP using horde credentials when not desired

[noreply at bugs.horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/12448
------------------------------------------------------------------------------
  Ticket             | 12448
  Updated By         | lfbm.andamentos at gmail.com
  Summary            | IMP using horde credentials when not desired
  Queue              | IMP
  Version            | 6.1.2
  Type               | Bug
  State              | Not A Bug
  Priority           | 1. Low
  Milestone          |
  Patch              |
  Owners             |
------------------------------------------------------------------------------


lfbm.andamentos at gmail.com (2013-07-18 00:12) wrote:

> I'm really not sure how much clearer we can make this.

You are assuming people will read the smtp parameters especifications  
after they already discarded the need of configuring smtp in IMP at all.

Also, the horde_auth explanation is a little ambiguous. It references  
horde's authentication, when actually it should reference horde  
mailer's authentication (in my case they differ). And when it says  
"those fields", it's not clear if those fields are the ones defined in  
horde mailer tab or in imp's backends.

Either way, it's confusing because you are led to think you do not  
need to set imp's smtp options when you are not overwriting horde  
mailer settings.

My suggestion would be something like this:

  * smtp: (array) These parameters will be used by IMP when
  *    sending email. If you want to use the default SMTP mailer
  *    already configured in Horde, you must set horde_auth
  *    parameter below. If you define any other parameters in this
  *    array, they will overwrite the default Horde SMTP parameters.


For the horde_auth explanation, I would suggest something like this:

  *     - horde_auth: (boolean) If true, populates the 'password' and
  *                   'username' parameters with Horde mailer's authentication
  *                   credentials (only if those fields are not  
defined in here).
  *                   Otherwise, IMP's authentication credentials are  
used instead
  *                   (this option only makes a difference if IMP's  
authentication
  *                   credenitals differ from Horde's).


It's just a suggestion and I'm sure it can be improved.

All of this must be very clear in your head because you wrote the  
program, but for those who just use it, I can assure you it's not that  
obvious and some clarification would certainly help. It's up to you.  
I'm just trying to contribute. Please receive my comments with an open  
spirit. Thanks.