On Fri, 2014-07-11 at 13:28 +0200, Michael Osipov wrote: > Am 2014-07-11 13:19, schrieb David Woodhouse: > > On Fri, 2014-07-11 at 13:04 +0200, Michael Osipov wrote: > >> Why do you provide a slash as a breaking char too? Backslash is the > >> only used char to separate domain from samaccountname. > > > > I didn't even look at that part — it's just in the context of my patch. > > > > I assume it's to allow people to specify the user on the command line > > with a slash instead of a backslash? > > Backslash isn't a problem as long as you do: > > $ curl --ntlm -u DOM\\michaelo http://...
I agree. In similar code elsewhere I have not chosen to support the use of a forward slash; only the backslash. However, this is what curl already did before I looked at it. Removing it now would have the potential to break existing users. Well, not that this code was working before I sent those two patches, so if you *really* want to make your case for removing the forward slash, I suppose that's fair enough. But you need to make it to someone other than me :) -- dwmw2
smime.p7s
Description: S/MIME cryptographic signature
------------------------------------------------------------------- List admin: http://cool.haxx.se/list/listinfo/curl-library Etiquette: http://curl.haxx.se/mail/etiquette.html
