Description: ------------ I wanted to set User-Agent HTTP header what I like. Submitted patch will use ini_get('HTTP_WebDAV_Client.UserAgent') .

Looks good at first glance. I will mention that a lot of other pear packages tend towards an $options array of some description in the constructor/factory method, it might be worth a look.

I'm sorry could not find a sample of $options array usage. Can you point one pear package, Daniel? Passing an option is difficult in this HTTP_WebDAV_Client package, because it is just a stream wrapper and we don't make an object directly. ....Ah, stream_context should be used !! I'll create another patch later.

I created a patch. The patch use stream context like this: require_once('HTTP/WebDAV/Client.php'); $c=stream_context_create(array('webdav'=>array('user_agent'=>'MyClient/0.1'))); $fp=file_get_contents('webdav://127.0.0.1/dav/test.txt',false,$c);

I read the PHP manual once again, and changed the patch to meet with the convention of http/https stream wrapper context option. http://www.php.net/manual/en/context.http.php

This bug has been fixed in CVS. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better.