Bug #76833 [Opn]: stream_socket_enable_crypto-win32.phpt fails
| From: | cmb@php.net | Date: | Sat, 01 Sep 2018 16:12:17 +0000 |
| Subject: | Bug #76833 [Opn]: stream_socket_enable_crypto-win32.phpt fails | ||
| References: | 1 | Groups: | php.bugs php.qa |
| Request: | Send a blank email to php-bugs+get-216843@lists.php.net to get a copy of this message | ||
Edit report at https://bugs.php.net/bug.php?id=76833&edit=1
ID: 76833
Updated by: cmb@php.net
Reported by: cmb@php.net
Summary: stream_socket_enable_crypto-win32.phpt fails
Status: Open
Type: Bug
Package: Testing related
Operating System: Windows
PHP Version: 7.2Git-2018-09-01 (Git)
Block user comment: N
Private report: N
New Comment:
Yes, master and 7.3 are apparently okay. Regarding the localized
message: it actually doesn't make sense to raise E_WARNING for
something that succeeded. Anyhow, now that I had a closer look at
the test[1], I don't think it makes much sense at all. Looks like
the test tries to increase code coverage, without testing the
actually relevant stuff (for instance, that after enabling crypto
the stream still âworksâ).
[1] <https://github.com/php/php-src/blob/php-7.3.0beta3/ext/standard/tests/streams/stream_socket_enable_crypto.phpt>
Previous Comments:
------------------------------------------------------------------------
[2018-09-01 15:01:48] kalle@php.net
It seems like Caruso merged this Windows specific test with the Unix counterpart for master and you
are right with the language specific error message, perhaps we should use %s in place there as the
error error messages seems to come from the function itself.
As for the error itself, that I did not have enough time to dig into here, but does the master one
pass?
------------------------------------------------------------------------
[2018-09-01 14:24:19] cmb@php.net
Description:
------------
ext\standard\tests\streams\stream_socket_enable_crypto-win32.phpt
fails with openssl-1.1.0i-vc15-x64 on PHP-7.2, as can be seen on
Appveyor[1] (I can also reproduce this locally). With older
openssl the test generally succeeds, but on a German system it
fails with the following diff:
012+ Warning: stream_socket_enable_crypto(): SSL: Der Vorgang wurde erfolgreich beendet.
012- Warning: stream_socket_enable_crypto(): SSL: The operation completed successfully.
[1] <https://ci.appveyor.com/project/php/php-src/build/PHP-7.2.build.8414/job/ay1uiffrme2rf5n8#L5512>
------------------------------------------------------------------------
--
Edit this bug report at https://bugs.php.net/bug.php?id=76833&edit=1