Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:80098
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: error (pb1.pair.com: domain bbroerman.net from 66.111.4.25 cause and error)
Message-ID: <54A6BD5A.60601@bbroerman.net>
Date: Fri, 02 Jan 2015 10:46:34 -0500
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:31.0) Gecko/20100101 Thunderbird/31.3.0
MIME-Version: 1.0
To: internals@lists.php.net, Wez Furlong <wez@thebrainroom.com>, 
 Daniel Lowrey <rdlowrey@php.net>,
 Chris Wright <daverandom@php.net>
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
Subject: Question about patch for bug 61285 - SSL connections do not timeout
From: bbroerman@bbroerman.net (Brad Broerman)

Hi,

   I have a pull request for issue 61285 - SSL connections do not
timeout, and have a question about it's status

   I created a pull request for this years ago, back in PHP 5.5.20 and
it was never merged into the master. I recently updated my clone of the
repo to the latest and re-wrote my patch against the newly updated
master and re-created the pull request.

   Is there a possibility of this being pulled? If not, why?  I have
tested this code both in the original 5.5.20, in the new 5.6.4, and
briefly with master. I believe it works pretty well.  In the pull
request itself, I added a comment about the travis ci build failing, but
I don't see where the failure has anything to do with this extension.
Would this block acceptance of this patch?


 Thanks
   Brad Broerman