Closing connection using TLS

Offen

#320 erstellt am 18.2.2025 4 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 4/5
Geschätzter Aufwand: 3-5 Tage
Anfängerfreundlichkeit: 45/100

Issue-Typ: Bug
Klarheit: Größtenteils klar
Aktivitätsstatus: Veraltet
Tech-Stack: php
Bereich: networking

Rechercherichtung

Start in src/Connection.php around the shutdown logic referenced by the issue, then reproduce the provided TLS server example. Compare the proposed fclose and stream_socket_enable_crypto approaches against PHP's TLS shutdown behavior; done means closing the connection cleanly without the current TLS shutdown problem.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

bug

When closing a connection on a server using TLS, eg:

<?php

require __DIR__ . '/vendor/autoload.php';

$server = new React\Socket\TcpServer($port = 9900);
$server = new React\Socket\SecureServer($server, null, [
    'local_cert' => 'server.pem',
    'passphrase' => 'secret'
]);

$server->on('connection', function (React\Socket\ConnectionInterface $connection) {
    echo '[' . $connection->getRemoteAddress() . ' connected]' . PHP_EOL;

    // do stuff
    
    $connection->end();
});

the connection is not closed properly, as https://github.com/reactphp/socket/blob/e04478a14b22c2f85697b917dcc26fa8f1654098/src/Connection.php#L123-L133
uses https://www.php.net/manual/en/function.stream-socket-shutdown.php.

Note comment https://www.php.net/manual/en/function.stream-socket-shutdown.php#125659 which states tls does not get shut down properly that way.

Solution seems to be to replace @\stream_socket_shutdown($this->stream, \STREAM_SHUT_RDWR); with \fclose($this->stream).

Other solution seems to be to call @\stream_socket_enable_crypto($this->stream, false); before the socket shutdown as mentioned in comment https://www.php.net/manual/en/function.stream-socket-shutdown.php#126303

Vorherrschende Sprache: PHP
Sterne: 1.3k
Forks: 160
PR-Merge-Kennzahlen: Keine gemergten PRs in 30 T.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus reactphp/socket

stream_socket_client throws fatal error if network is unreachable Offen

question

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 45/100

reactphp/socket#328 · 2 Kommentare · Erstellt am 24.10.2025
Roadmap to reactphp/socket v3 Offen

maintenance

Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 20/100

reactphp/socket#312 · 3 Reaktionen · Erstellt am 09.2.2024
Allow getRemoteAddress to return the PID of a connecting Unix connection Offen

help wanted new feature

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 45/100

reactphp/socket#150 · 9 Kommentare · Erstellt am 13.2.2018
Auto-generate server certs Offen

help wanted new feature

Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 20/100

reactphp/socket#95 · 1 Kommentar · Erstellt am 23.4.2017
Support STARTTLS Offen

help wanted new feature

Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 25/100

reactphp/socket#89 · 3 Kommentare · Erstellt am 10.4.2017

Alle Issues in reactphp/socket