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

Tests need adaptation to work with Netty 5 changes and the new Buffer API #2303

Merged
merged 1 commit into from
Jun 17, 2022

Conversation

violetagg
Copy link
Member

WebsocketTest#firefoxConnectionTest

https://datatracker.ietf.org/doc/html/rfc6455#section-4.1

The request MUST include a header field with the name Sec-WebSocket-Version. The value of this header field MUST be 13.

The request MUST include a header field with the name Sec-WebSocket-Key. The value of this header field MUST be a nonce consisting of a randomly selected 16-byte value that has been base64-encoded (see Section 4 of [RFC4648]). The nonce MUST be selected randomly for each connection.

… API

WebsocketTest#firefoxConnectionTest

https://datatracker.ietf.org/doc/html/rfc6455#section-4.1
`The request MUST include a header field with the name Sec-WebSocket-Version.  The value of this header field MUST be 13.`

`The request MUST include a header field with the name Sec-WebSocket-Key.  The value of this header field MUST be a
nonce consisting of a randomly selected 16-byte value that has been base64-encoded (see Section 4 of [RFC4648]).
The nonce MUST be selected randomly for each connection.`
Copy link
Member

@pderop pderop left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@violetagg violetagg marked this pull request as ready for review June 17, 2022 08:50
@violetagg violetagg added the type/enhancement A general enhancement label Jun 17, 2022
@violetagg violetagg added this to the 2.0.0-M1 milestone Jun 17, 2022
@violetagg
Copy link
Member Author

@pderop Thanks for the review

@violetagg violetagg merged commit c284c4f into netty5 Jun 17, 2022
@violetagg violetagg deleted the failing-tests branch June 17, 2022 08:59
@violetagg
Copy link
Member Author

Related to #1873

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/enhancement A general enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants