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

* Your Email Address
* Spam protection	Enter the letters below: .__ ..___\ /. . [ __ \|[__ >< \|_/ [_./\__\|\| / \\| \
Comment	>> 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.
Attachment
Watch this ticket

Saved Queries