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

WIP: Test for TLS websockets #1

Closed
wants to merge 5 commits into from
Closed

Conversation

jflatow
Copy link

@jflatow jflatow commented Oct 5, 2023

This is meant to be a test for denoland#18977, but encountering a different issue on the potentially fixed branch, at the moment.

A bit stuck as the client here gets an 'InvalidData: invalid data' error right away. Opening a WebSocket from the browser to the server running in the test is fine. And the test passes if the cert/key are commented out and wss -> ws.

mmastrac and others added 5 commits October 5, 2023 10:25
This is meant to be a test for denoland#18977, but encountering a different issue on the potentially fixed branch, at the moment.

A bit stuck as the client here gets an 'InvalidData: invalid data' error right away.
Opening a WebSocket from the browser to the server running in the test is fine.
And the test passes if the cert/key are commented out and wss -> ws.
@jflatow
Copy link
Author

jflatow commented Oct 5, 2023

(Also, I should mention I've tried with and without the setTimeout, and increasing the timeout to up to 90s, but it doesn't change anything)

@guest271314
Copy link

Nothing changed on deno 1.37.1. The WSS server is still firing error event handler in the server

No response from ping frame..

@mmastrac
Copy link
Owner

Pulling this test into denoland#20518 and closing this one. Thanks!

@mmastrac mmastrac closed this Oct 30, 2023
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.

3 participants