swooletw / laravel-swoole

High performance HTTP server based on Swoole. Speed up your Laravel or Lumen applications.
MIT License
4.04k stars 390 forks source link

isServerWebsocket method has a bug~ #504

Closed pslxx closed 2 years ago

pslxx commented 2 years ago

Make sure you read Issues Guideline and answer these questions before submitting your issue. Thanks! (Any non-English issues will be closed immediately.)

  1. Please provide your PHP and Swoole version. (php -v and php --ri swoole)

PHP 7.3.24-3+ubuntu18.04.1+deb.sury.org+1 (cli) (built: Oct 31 2020 16:59:59) ( NTS )

swoole Version => 4.5.10

  1. Please provide your Laravel/Lumen version. 8.14

  2. Which release version of this package are you using?

2.6.68

  1. What did you do? If possible, provide a recipe for reproducing the error.

enable websocket an test a http request

  1. What did you expect to see?

  2. What did you see instead?

Http Reuqest fired onClose hook

isServerWebsocket return true,,but its a http request ,not websocket connect

if a http request fd websocket_status value is "0",but key exists

image

image image