Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

EventSub Over WebSocket #290

Open
Nerixyz opened this issue Nov 1, 2022 · 3 comments · Fixed by #291
Open

EventSub Over WebSocket #290

Nerixyz opened this issue Nov 1, 2022 · 3 comments · Fixed by #291
Assignees
Labels
enhancement New feature or request

Comments

@Nerixyz
Copy link
Contributor

Nerixyz commented Nov 1, 2022

Twitch announced, that EventSub over WebSockets is now in public beta. This repo should implement the structures and requests to parse and manage the subscriptions and events.

Maybe there should be another project under the twitch-rs org that implements a client? Or more generally a project that has EventSub helpers for web-frameworks and a WebSocket client?

@Nerixyz Nerixyz added the enhancement New feature or request label Nov 1, 2022
@Emilgardis Emilgardis self-assigned this Nov 1, 2022
@Emilgardis Emilgardis linked a pull request Nov 1, 2022 that will close this issue
@Emilgardis
Copy link
Member

Emilgardis commented Nov 1, 2022

the websocket rust saga is horrible, tungstenite seems to be the agreed upon base though.

How would a websocket client with twitch_api integration work?

for web-frameworks I can clearly see the benefits, as explained by you previously, but for websockets I'm unsure.

for pubsub currently, I do https://github.com/Emilgardis/twitch_discord_moderation/blob/main/src/subscriber.rs#L189

I'm not sure I see how to improve it with another tungstenite abstraction

@bors bors bot closed this as completed in #291 Dec 4, 2022
@Emilgardis Emilgardis reopened this Dec 4, 2022
@Emilgardis
Copy link
Member

Reopening this as providing more functionality for clients remains open

@Emilgardis
Copy link
Member

Should prioritize this as .message is probably around the corner now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants