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

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 ( 
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 ?


Red Hat Virtualization group
Daniel Veillard      | virtualization library
veillard redhat com  | libxml GNOME XML XSLT toolkit | Rpmfind RPM search engine

[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]