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