remove all pre-existing upgrade listeners

fastify / fastify-websocket

basic websocket support for fastify

MIT License

394 stars 72 forks source link

remove all pre-existing upgrade listeners #253

Closed mcollina closed 1 year ago

mcollina commented 1 year ago

Fixes #249

Checklist

[x] run npm run test and npm run benchmark
[x] tests and/or benchmarks are included
[x] documentation is changed or added
[x] commit message and code follows the Developer's Certification of Origin and the Code of conduct

mcollina commented 1 year ago

This is a complete rewrite of the shutdown logic to avoid monkey-patching. It's a semver-major change. This also includes a behavior change now, as websocket requests received while the server is closing will now receive a 503 like all other requests.