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 <greearb_at_candelatech.com>
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