Re: [xml] [PATCH] xmlIO HTTPS handler using libcurl [version 2]

[joel reed <joelwreed gmail com>]

Daniel Veillard wrote:
> On Wed, Mar 21, 2007 at 01:38:43PM -0400, joel reed wrote:
>   
> > Yes I agree another dependency is annoying. We are doing forms based 
> > authentication over SSL here, so the need to handle http basic/digest 
> > type authentication hasn't come up but I can see the usefulness for others.
> >     
>
>   I see,
>
>   
> > One thing I had considered is making the curl patch handle http, https, 
> > ftp, sftps, and whatever else (http://curl.haxx.se/docs/features.html) 
> > libcurl can handle that would be of interest for xmlIO. As part of that 
> > I could add an authentication callback.
> >
> > Could we say that if you enable curl support,  nano http/ftp handlers 
> > are disabled at compile time or would you like to approach it some other 
> > way? (if the above sounds interesting) If the dependency is really too 
> > annoying, no problem, patch can sit on mailing list for others if they 
> > need it.
> >     
>
>   I tend to think that if we integrate curl support then let's do it
> fully, and avoid the nano side of things. Then the configure option 
> would really be useful to a larger audience.
>   At this point my take is that an optional feature plugging curl fully
> replacing the nano modules and adding authentication would makes a
> lot more sense than a limited plug, and that should be added to the 
> default code base because a lot more users are likely to use/debug/fix
> potential problems.
>
>    Makes sense ?
>
> Daniel
>
>   
This does make sense Daniel, thanks for thinking through the right 
approach to take.
I'll take a crack at a v3 patch - give me some time.

jr