Req #67224 [Asn->Csd]: stream_socket_enable_crypto() requires crypto type when enabling

Edit report at https://bugs.php.net/bug.php?id=67224&edit=1 ID: 67224 Updated by: rdlowrey@php.net Reported by: daverandom@php.net Summary: stream_socket_enable_crypto() requires crypto type when enabling -Status: Assigned +Status: Closed Type: Feature/Change Request Package: Streams related Operating System: Any PHP Version: 5.6.0beta2 Assigned To: rdlowrey Block user comment: N Private report: N New Comment: PR607 has been merged and this should now be fixed in PHP-5.6 and master. The changes will be reflected in the 5.6beta3 release. Previous Comments: ------------------------------------------------------------------------ [2014-05-07 10:20:19] daverandom@php.net Description: ------------ Since 5.6.x the crypto type can be determined from the ssl->crypto_method stream context value, it should no longer be a required argument. An error should emitted only when neither value is supplied. Test script: --------------- $sock = stream_socket_client('tcp://google.com:443'); stream_context_set_option($sock, 'ssl', 'crypto_method', STREAM_CRYPTO_METHOD_TLS_CLIENT); stream_socket_enable_crypto($sock, true); Expected result: ---------------- Crypto enabling succeeds, with method determined from the stream's context options Actual result: -------------- Crypto enabling fails because 3rd argument is not supplied ------------------------------------------------------------------------ -- Edit this bug report at https://bugs.php.net/bug.php?id=67224&edit=1