unjs / crossws

🔌 Cross-platform WebSocket Servers for Node.js, Deno, Bun and Cloudflare Workers.
https://crossws.unjs.io
Other
395 stars 17 forks source link

refactor!: remove adapter hooks #72

Closed pi0 closed 3 months ago

pi0 commented 3 months ago

The initial idea behind adapter hooks was to allow an escape hatch over abstractions.

During development and integrations, they actually wasn't useful at all and current ~33 additional hooks could only make runtime slower, besides they were not resolvable so not compatible with application routers (and in top level users anyway have power to hook into more)

This PR removes them but we might add back some adapter specific hooks under normal hooks if needed.

cloudflare-workers-and-pages[bot] commented 3 months ago

Deploying crossws with  Cloudflare Pages  Cloudflare Pages

Latest commit: 00dfc1c
Status:🚫  Build failed.

View logs