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.

Sign up for GitHub

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

Jump to bottom

Require auth for /listen API #1236

Open
beastoin opened this issue Nov 2, 2024 · 2 comments
Open

Require auth for /listen API #1236

beastoin opened this issue Nov 2, 2024 · 2 comments

Comments

@beastoin
Copy link
Collaborator

beastoin commented Nov 2, 2024 •
edited
Loading

I don't want anyone be able to access to the socket with a simple call wscat to /v2/listen API.

please add auth to that api
@beastoin
Copy link
Collaborator Author

beastoin commented Nov 2, 2024

@nquang29 welcome to the party. yours.

@vincentkoc vincentkoc mentioned this issue Nov 3, 2024
Open
@vincentkoc
Copy link
Contributor

vincentkoc commented Nov 3, 2024 •
edited
Loading

FYI possible breaking changes on #1238 - easier to add my changes as some of the auth is tested there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
omi TODO
Status: No status
Milestone
No milestone
Development

No branches or pull requests
2 participants
@vincentkoc @beastoin