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

Feature/http3 support 5.x #5311

Open
wants to merge 99 commits into
base: master
Choose a base branch
from

Conversation

imz87
Copy link

@imz87 imz87 commented Sep 10, 2024

Motivation
This PR introduces support for HTTP/3 in Vert.x, leveraging the existing HTTP/3 facilities provided by Netty. The motivation behind this change is to bring the latest HTTP protocol version to Vert.x, allowing developers to take advantage of the performance improvements and new features that HTTP/3 offers, such as reduced latency, improved security, and better handling of packet loss.

Currently, the PR focuses on adding HTTP/3 support for the client side only. This is the first step towards full HTTP/3 integration in Vert.x. The server-side implementation will be addressed in a subsequent PR, and comprehensive testing will be conducted at that stage.

By enabling HTTP/3 support on the client side, this change positions Vert.x to remain a competitive and modern toolkit for reactive applications. It allows developers to begin experimenting with and utilizing HTTP/3 in their client applications, paving the way for broader adoption once server-side support is complete.

Co-Authored: @zekronium

@imz87 imz87 mentioned this pull request Sep 10, 2024
@vietj vietj added this to the 5.1.0 milestone Sep 11, 2024
@zekronium
Copy link
Contributor

@vietj It should be good for review and open for changes if needed

…upport-5.x

� Conflicts:
�	vertx-core/src/main/java/io/vertx/core/http/impl/Http1xServerConnection.java
�	vertx-core/src/main/java/io/vertx/core/http/impl/Http2ServerResponse.java
@vietj
Copy link
Member

vietj commented Oct 28, 2024

@zekronium for now it is on hold until HTTP/3 is part of Netty 4.2 branch

@zekronium
Copy link
Contributor

@vietj Why not have it as an optional dependency. Might take while, whilst other projects are already using it such as netty reactor

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