Bug #61285 [Opn->Csd]: SSL connections do not timeout
| From: | rdlowrey@php.net | Date: | Wed, 04 Mar 2015 15:58:35 +0000 |
| Subject: | Bug #61285 [Opn->Csd]: SSL connections do not timeout | ||
| References: | 1 | Groups: | php.bugs |
| Request: | Send a blank email to php-bugs+get-191143@lists.php.net to get a copy of this message | ||
Edit report at https://bugs.php.net/bug.php?id=61285&edit=1
ID: 61285
Updated by: rdlowrey@php.net
Reported by: tony2001@php.net
Summary: SSL connections do not timeout
-Status: Open
+Status: Closed
Type: Bug
Package: OpenSSL related
PHP Version: 5.4SVN-2012-03-05 (SVN)
-Assigned To:
+Assigned To: rdlowrey
Block user comment: N
Private report: N
New Comment:
This has been addressed in upstream 5.5/5.6/master branches. Thanks to bbroerman for the associated
patches.
http://git.php.net/?p=php-src.git;a=commitdiff;h=fd4641696cc67fedf494717b5e4d452019f04d6f
http://git.php.net/?p=php-src.git;a=commitdiff;h=1482ed2d5660c3875add40706a18fe29e2b3ff70
http://git.php.net/?p=php-src.git;a=commitdiff;h=dddbe0fc338a0f01ba336e84755694fb9bfbeb53
Previous Comments:
------------------------------------------------------------------------
[2014-12-31 04:44:19] bbroerman at bbroerman dot net
The comparison to the baseline (php/php-src) yielded less errors on my fork than on the origin. The
php/php-src had the same errors that mine did, and more.
I will do functionality testing tomorrow, and if it works well, I will create a pull request
------------------------------------------------------------------------
[2014-12-31 04:09:24] bbroerman at bbroerman dot net
I am making another run at the fix, this time updated for the current trunk release of PHP (my
previous was for 5.5.18)
I am trying to test, but I am running into issues with make test that I am not sure are related or
not. When I know for sure I'll add another comment.
If they are related, I may need guidance on what these tests are, and how timeouts on read/write may
affect them.
------------------------------------------------------------------------
[2013-08-24 14:59:58] boen dot robot at gmail dot com
At the risk of being annoying, I'd like to repeat what I said in GitHub, in case it was
missed...
bbroerman's changes have been done already. So, could this be merged now please?
(This issue also affects me, obviously...)
------------------------------------------------------------------------
[2013-01-23 03:05:04] bbroerman at bbroerman dot net
When I get a chance, I'll back out the changes, and re-do them with the same whitespace as the
original... May be a couple weeks.
------------------------------------------------------------------------
[2013-01-23 02:40:49] bbroerman at bbroerman dot net
I sent the pull request a couple days ago.
------------------------------------------------------------------------
The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at
https://bugs.php.net/bug.php?id=61285
--
Edit this bug report at https://bugs.php.net/bug.php?id=61285&edit=1