On 03/26/2010 07:49 AM, Daniel Stenberg wrote:
On Thu, 25 Mar 2010, Ben Greear wrote:

And some other mail URLs as well. Seems to me that all could be
enabled, and just let protocols ignore the username/password if they
don't care. This patch just adds a few, however. I've only tested this
with SMTP and SMTPS, but can't imagine it causing harm to the others.

mail: Allow mail URLs to contain username/passwords.

There's no need to include the *S versions of the protocols in that
check, as all *S protocol versions (POP3S, IMAPS etc) of the handler
struct also have the non-S bits set in their 'protocol' bitmasks.

Otherwise I'm fine with allowing this.

Could we just change the code to parse user:passwd for ALL protocols
and let protocols ignore the user:passwd as needed?

Thanks,
Ben


--
Ben Greear <[email protected]>
Candela Technologies Inc  http://www.candelatech.com
-------------------------------------------------------------------
List admin: http://cool.haxx.se/list/listinfo/curl-library
Etiquette:  http://curl.haxx.se/mail/etiquette.html

Reply via email to