Description: ------------ This class has no method for closing connection. I think that new disconnect() method is needed. Or, if Connection:KeepAlive will not be support, simply sendRequest() method may close socket.

If you have many requests with HTTP_Request, the count of file descriptor grow to big. Finally, it will reach to process limit, error returns. It is simple patch: --- Request.php.orig Thu Jul 1 21:44:19 2004 +++ Request.php Thu Jul 1 21:46:20 2004 @@ -619,7 +619,8 @@ } elseif ($this->_allowRedirects AND $this->_redirects > $this->_maxRed irects) { return PEAR::raiseError('Too many redirects'); } - + + $this->_sock->disconnect(); return true; }

Dear Maintainer, Your package http_request has currently 2 open bugs. We urge you to take the necessary steps to solve the reported issues at your soonest convenience. If the bug issue hasn’t been addressed yet you are kindly asked to take the necessary steps to ensure a prompt resolution of the problem. If you already addressed the reported problem feel free to change the bug status as soon as possible. Regards David Costa PEAR Quality Assurance pear-qa@lists.php.net

Since Keep-Alive is not supported, an easiest solution would be to add a disconnect() call, as suggested by koyama at hoge dot org. Will do soon.

I found this bug too today, it broke our nice PEAR IRC bot (PEARgirl) on #pear efnet. Since it runs as a daemon the script never ends, each request will make a new connection. After a week we got over 1000 not closed connections: root@server-4:~# netstat -plan|grep php|wc -l 1020 so new connections are not possible from this script... I think this bug is very important to fix, its something like memory leaks. I am changing the bug-type to bug.

Fixed in CVS