Description: ------------ So, here's my use case: I want to send multiple Link header along with a request. From what I understand through the spec is, that this is possible on Link and Accept headers by combining them, or probably even with: Link: foo Link: bar ... HTTP_Request2::setHeader() fails to honor that. Here's a patch to correct it. Test script: --------------- <?php // here's a test case public function testHttpRequest2Bug() { $req = new HTTP_Request2; $req->setUrl('http://localhost/foo/bar'); $req->setHeader('Link: foo'); $req->setHeader('Link: bar'); $req->setAdapter('mock'); $response = $req->send(); $headers = $req->getHeaders(); $links = $headers['link']; $count = explode(',', $links); $this->assertEquals(count($headers), 2); // Link headers combined + 1 for user-agent $this->assertEquals(count($count), 2); } ?>

Can't upload the patch, here is a link: http://friendpaste.com/5MaInyX2VA1lAJM88QaCKv

This bug has been fixed in SVN. 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. There is now a third $replace parameter in setHeader() that controls whether a new value will overwrite an old one or be appended to it.