Dear all,

While using neon for querying WebDAV repository I discover that the
ne_simple_propfind doesn't handle namespace nor properties (both are
simply ignored by the function). As this behaviour is not explicitly
documented in the code, this can cause some unexpected results.

Attached to this mail is a proposed solution so that at namespace and
properties are no longer ignored by the function. When namespace and
properties are not used, this should have only minor impact on the
performance: 
the 
        ne_buffer_concat(hdl->value, "<", name, ">", NULL);
is replaced by
      ne_buffer_concat(hdl->value, "<", name, NULL);
      ne_buffer_concat(hdl->value, ">", NULL);
And 3 additional ifs are performed.

I'm not sure if the way I implemented the path is the best solution, but
it was the less intrusive I found. Anyway I would be happy to improve it
if there is interest.

Hope this is useful,

Best regards

Julien


Attachment: ne_props_properties_namespace.patch
Description: ne_props_properties_namespace.patch

_______________________________________________
neon mailing list
[email protected]
http://mailman.webdav.org/mailman/listinfo/neon

Reply via email to