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

Setup h3spec in CI #110

Open
seanmonstar opened this issue Aug 2, 2022 · 1 comment
Open

Setup h3spec in CI #110

seanmonstar opened this issue Aug 2, 2022 · 1 comment
Labels
E-medium Effort: medium. Some knowledge of how the internals work would be useful.

Comments

@seanmonstar
Copy link
Member

Similar to how we check h2spec in h2, it could be a good idea to set up a CI job to check the library with h3spec (or alternatives if they are more complete).

@seanmonstar seanmonstar added the E-medium Effort: medium. Some knowledge of how the internals work would be useful. label Aug 2, 2022
@Ruben2424 Ruben2424 mentioned this issue Aug 7, 2022
@Ruben2424
Copy link
Contributor

I have done the CI Setup in #112. But some tests get skiped because these are failing.

The skiped:

        --skip "/QUIC servers/MUST send PROTOCOL_VIOLATION if CRYPTO in 0-RTT is received [TLS 8.3]/" \
        --skip "/QUIC servers/MUST send TRANSPORT_PARAMETER_ERROR if original_destination_connection_id is received [Transport 18.2]/" \
        --skip "/QUIC servers/MUST send TRANSPORT_PARAMETER_ERROR if retry_source_connection_id is received [Transport 18.2]/" \
        --skip "/QUIC servers/MUST send PROTOCOL_VIOLATION on no frames [Transport 12.4]/" \
        --skip "/HTTP/3 servers/MUST send H3_MESSAGE_ERROR if a pseudo-header is duplicated [HTTP/3 4.1.1]/" \
        --skip "/HTTP/3 servers/MUST send H3_FRAME_UNEXPECTED if CANCEL_PUSH is received in a request stream [HTTP/3 7.2.5]/" \
        --skip "/HTTP/3 servers/MUST send QPACK_ENCODER_STREAM_ERROR if a new dynamic table capacity value exceeds the limit [QPACK 4.1.3]/" \
        --skip "/HTTP/3 servers/MUST send QPACK_DECODER_STREAM_ERROR if Insert Count Increment is 0 [QPACK 4.4.3]/" \

Some of them may be outdated. But some need to be fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
E-medium Effort: medium. Some knowledge of how the internals work would be useful.
Projects
None yet
Development

No branches or pull requests

2 participants