Skip to content

remove OpenSSL backend#2458

Merged
ghedo merged 1 commit into
masterfrom
rm-openssl
May 5, 2026
Merged

remove OpenSSL backend#2458
ghedo merged 1 commit into
masterfrom
rm-openssl

Conversation

@ghedo
Copy link
Copy Markdown
Member

@ghedo ghedo commented May 5, 2026

The OpenSSL backend (the openssl feature, backed by quictls/openssl) was incomplete: 0-RTT and custom private key signing were unsupported, several tests were disabled under it, and the underlying quictls fork is unmaintained.

The BoringSSL backend code is left as-is. Comments, FFI declarations and the tls/, crypto/ module layout still reflect a multi-backend setup. That can be cleaned up later without touching protocol code.

@ghedo ghedo requested a review from a team as a code owner May 5, 2026 11:17
@ghedo ghedo mentioned this pull request May 5, 2026
Base automatically changed from ci-cross-images to master May 5, 2026 13:01
The OpenSSL backend (the `openssl` feature, backed by quictls/openssl)
was incomplete: 0-RTT and custom private key signing were unsupported,
several tests were disabled under it, and the underlying quictls fork
is unmaintained.

The BoringSSL backend code is left as-is. Comments, FFI declarations
and the `tls/`, `crypto/` module layout still reflect a multi-backend
setup. That can be cleaned up later without touching protocol code.
@ghedo ghedo enabled auto-merge (rebase) May 5, 2026 13:02
auto-merge was automatically disabled May 5, 2026 13:10

Branch protection rule check failed

@ghedo ghedo merged commit a6c4dff into master May 5, 2026
40 checks passed
@ghedo ghedo deleted the rm-openssl branch May 5, 2026 13:11
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