FormidableLabs / envy

Node.js Telemetry & Network Viewer
https://envy-webui.vercel.app
MIT License
109 stars 0 forks source link

Consolidate websocket client into envyjs/core #121

Closed carbonrobot closed 1 year ago

carbonrobot commented 1 year ago

Uses isomorphic-ws to consolidate the Websocket client into envyjs/core so we don't maintain two different implementations for this.

Also improves the Retry timing. previously we used a function that scaled too quickly and resulted in a delay of 90mins, which is impractical. The new delay timing has a flatter curve and a max delay of 5mins with 30 max attempts. We also change the log to debug level so it doesn't spam the logs.

Delay graph before

image

Delay graph after

image

Fixes #119 Fixes #120

changeset-bot[bot] commented 1 year ago

🦋 Changeset detected

Latest commit: 189d6d6b2787f71325b44bf1738346dea52e48f3

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 6 packages | Name | Type | | -------------- | ----- | | @envyjs/core | Minor | | @envyjs/node | Minor | | @envyjs/web | Minor | | @envyjs/webui | Minor | | @envyjs/apollo | Minor | | @envyjs/nextjs | Minor |

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR