Skip to content

Commit

Permalink
Remove unnecessary getApplicationLayerProtocols() method from HttpDri…
Browse files Browse the repository at this point in the history
…verMiddleware
  • Loading branch information
bwoebi committed Feb 15, 2024
1 parent 6dd6e27 commit c23e971
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 7 deletions.
5 changes: 0 additions & 5 deletions src/Driver/HttpDriverMiddleware.php
Original file line number Diff line number Diff line change
Expand Up @@ -13,9 +13,4 @@ public function createHttpDriver(
ErrorHandler $errorHandler,
Client $client,
): HttpDriver;

/**
* @return list<string>
*/
public function getApplicationLayerProtocols(HttpDriverMiddleware $next): array;
}
4 changes: 2 additions & 2 deletions src/SocketHttpServer.php
Original file line number Diff line number Diff line change
Expand Up @@ -197,8 +197,8 @@ public function expose(SocketAddress|string $socketAddress, BindContext|QuicServ
$socketAddress = SocketAddress\fromString($socketAddress);
}

$name = $socketAddress->toString();
if (isset($this->addresses[$name]) && ($bindContext instanceof QuicServerConfig) == ($this->addresses[$name][1] instanceof QuicServerConfig)) {
$name = ($bindContext instanceof QuicServerConfig ? "quic-" : "") . $socketAddress->toString();
if (isset($this->addresses[$name])) {
throw new \Error(\sprintf('Already exposing %s on HTTP server', $name));
}

Expand Down

0 comments on commit c23e971

Please sign in to comment.