Edit report at https://bugs.php.net/bug.php?id=77938&edit=1

 ID:                 77938
 Updated by:         c...@php.net
 Reported by:        v-altruo at microsoft dot com
 Summary:            socket_get_option error
 Status:             Verified
 Type:               Bug
-Package:            Streams related
+Package:            Testing related
 Operating System:   Windows
 PHP Version:        7.2.18RC1
-Assigned To:        
+Assigned To:        cmb
 Block user comment: N
 Private report:     N

 New Comment:

Thanks for reporting!  As requinix said, the test has to be fixed
to cater to the recently introduced HTTPS redirect of www.php.net
(the warning is suppressed due to the @ operator).


Previous Comments:
------------------------------------------------------------------------
[2019-04-24 21:11:38] requi...@php.net

Warning: socket_import_stream(): cannot represent a stream of type 
tcp_socket/ssl as a Socket Descriptor

Broken by the new HTTPS redirect. Adding
  "http" => ["follow_location" => 0]
to the options array works.

------------------------------------------------------------------------
[2019-04-24 21:07:43] v-altruo at microsoft dot com

Description:
------------
Failed regardless of OPCache being enabled or disabled and if it was TS or NTS.
Test file location: 
ext\standard\tests\stream\stream_context_tcp_nodelay_fopen.phpt

Test script:
---------------
$ctxt = stream_context_create([
        "socket" => [
                "tcp_nodelay" => true
        ]
]);

$stream = fopen("http://www.php.net";, "r", false,  $ctxt);

$socket =
        @socket_import_stream($stream);

var_dump(socket_get_option($socket, STREAM_IPPROTO_TCP, TCP_NODELAY) > 0);

Expected result:
----------------
bool(true)

Actual result:
--------------
Warning: socket_get_option() expects parameter 1 to be resource, boolean given 
in... 
bool(false)


------------------------------------------------------------------------



--
Edit this bug report at https://bugs.php.net/bug.php?id=77938&edit=1

Reply via email to