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

Randomize payload when using cleartext WebSocket #332

Closed
bassosimone opened this issue Jun 2, 2021 · 3 comments
Closed

Randomize payload when using cleartext WebSocket #332

bassosimone opened this issue Jun 2, 2021 · 3 comments
Labels
enhancement

Comments

@bassosimone
Copy link
Contributor

We should randomize the payload when using cleartext WebSocket. A bunch of zeroes is too easy to compress.
@bassosimone bassosimone mentioned this issue Jun 2, 2021
Closed
@nosnilmot
Copy link
Contributor

I was about to have a go at implementing this, but it looks to me like it is already using random data here, am I missing something?

@laiyi-ohlsen
Copy link

@bassosimone Do you have thoughts on the above? Is this already implemented?

@robertodauria
Copy link
Contributor

As far as I can tell, this is implemented already.

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

No branches or pull requests
4 participants
@robertodauria @bassosimone @laiyi-ohlsen @nosnilmot