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 4.4 #5287

Closed
wants to merge 99 commits into from

Conversation

imz87
Copy link

@imz87 imz87 commented Aug 21, 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.

@vietj
Copy link
Member

vietj commented Aug 27, 2024

@imz87 can you rebase your PR on the master branch of Vert.x instead ? the 4.4 branch will not get any releases

@imz87
Copy link
Author

imz87 commented Aug 27, 2024

@imz87 can you rebase your PR on the master branch of Vert.x instead ? the 4.4 branch will not get any releases

@vietj Yes, you’re right. I will rebase my branch. Which branch should I rebase onto, branch 4.x or branch 5.x?

@vietj
Copy link
Member

vietj commented Aug 28, 2024

it will go to master branch 5.x

@imz87 imz87 closed this Aug 29, 2024
@imz87 imz87 reopened this Aug 29, 2024
@imz87
Copy link
Author

imz87 commented Aug 29, 2024

it will go to master branch 5.x

Thanks a lot for your response. I will merge with 5.x

@vietj
Copy link
Member

vietj commented Aug 29, 2024

can you instead perform a clean rebase @imz87 ?

@imz87
Copy link
Author

imz87 commented Aug 29, 2024

can you instead perform a clean rebase @imz87 ?

@vietj sure, I will rebase onto 5.x!

@vietj
Copy link
Member

vietj commented Aug 29, 2024

thank you, I will follow up closely

@zekronium
Copy link
Contributor

No chance of this landing on 4.x too? What is the ETA for 5.x?

@vietj
Copy link
Member

vietj commented Aug 29, 2024

we plan to release 5.0 after Netty 4.2 is released scheduled mid october

@vietj
Copy link
Member

vietj commented Aug 29, 2024

in netty 4.1, http3 is incubator so it owuld be a separate project (becuase incubation), in netty 4.2 http3 will be in the main netty distribution

@imz87 imz87 closed this Sep 10, 2024
@imz87
Copy link
Author

imz87 commented Sep 10, 2024

@vietj

I've opened a new PR for the 5.x branch. you can find it here: #5311

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