Skip to content

Conversation

@maxfortun
Copy link

Issue [#162].

Feature:
Dynamically provide Protocols on reconnect.

Use case:
Protocols is the only mechanism available to WebSocket to pass initialization information to the backend at connection time. As such, this could be leveraged for authorization. Modern authorization mechanisms, such as tokens, rotate over time and reconnecting with an old token will eventually fail. A dynamic Protocols provider can produce valid tokens at reconnect time.

threepointone added a commit to partykit/partykit that referenced this pull request Jan 8, 2023
This forks `reconnecting-websocket` into our own package, and adds some functionality from abandoned PRs on the source repo. Of note:
- pladaria/reconnecting-websocket#166 Fix: handle error if getNextUrl throws
- pladaria/reconnecting-websocket#132 feat: make protocols updatable
- pladaria/reconnecting-websocket#141 [Fix] Socket doesn't connect again after closing while connecting

(TODO: more)
- pladaria/reconnecting-websocket#163 Support for Dynamic Protocols
- pladaria/reconnecting-websocket#47 reconnecting and reconnectscheduled custom events
threepointone added a commit to partykit/partykit that referenced this pull request Jan 8, 2023
This forks `reconnecting-websocket` into our own package, and adds some functionality from abandoned PRs on the source repo. Of note:
- pladaria/reconnecting-websocket#166 Fix: handle error if getNextUrl throws
- pladaria/reconnecting-websocket#132 feat: make protocols updatable
- pladaria/reconnecting-websocket#141 [Fix] Socket doesn't connect again after closing while connecting

(TODO: more)
- pladaria/reconnecting-websocket#163 Support for Dynamic Protocols
- pladaria/reconnecting-websocket#47 reconnecting and reconnectscheduled custom events
@diliks
Copy link

diliks commented Apr 22, 2025

@maxfortun, hi, could you pls sign CLI on opensumi#11 ?

@maxfortun
Copy link
Author

@diliks, sorry, that whole workflow looks very sus to me. Feel free to copy paste my changes and take ownership if you like.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants