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

HTTP/2 - Enable H2 protocol #490

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

Conversation

NiccoMlt
Copy link
Contributor

@NiccoMlt NiccoMlt commented Aug 30, 2024

Closes #410 and, along with #409, closes #181
Closes #506

@NiccoMlt NiccoMlt linked an issue Aug 30, 2024 that may be closed by this pull request
@NiccoMlt NiccoMlt force-pushed the 410-http2-enable-http2-h2 branch from def579f to 12cdd0c Compare September 24, 2024 06:56
@NiccoMlt NiccoMlt force-pushed the 410-http2-enable-http2-h2 branch from 9fa84aa to c4903ab Compare October 3, 2024 13:18
@NiccoMlt NiccoMlt force-pushed the 410-http2-enable-http2-h2 branch from 16c6b5c to b9d4c7b Compare October 20, 2024 15:36
@NiccoMlt NiccoMlt changed the title 410 http2 enable http2 h2 HTTP/2 - Enable H2 protocol Oct 20, 2024
@NiccoMlt NiccoMlt self-assigned this Oct 20, 2024
@hamadodene
Copy link
Contributor

@NiccoMlt Take a look on this issue #506
I also get this error:

Oct 28, 2024 9:20:55 AM org.carapaceproxy.core.ListeningChannel map
SEVERE: Error booting certificate for SNI hostname cara17test.xxx.it, on listener NetworkListenerConfiguration[host=0.0.0.0, port=4089, ssl=true, sslCiphers=, defaultCertificate=*, sslProtocols=[TLSv1.3], soBacklog=128, keepAlive=true, keepAliveIdle=300, keepAliveInterval=60, keepAliveCount=8, maxKeepAliveRequests=10, forwardedStrategy=IF_TRUSTED, trustedIps=[127.0.0.1], protocols=[H2], group=DefaultChannelGroup(name: group-0x2, size: 0)]

The certificato of domain cara17test.xxx.it is available. I don't have the full stack. I will try to debug it later

@NiccoMlt NiccoMlt force-pushed the 410-http2-enable-http2-h2 branch from b9d4c7b to 9f8ad17 Compare October 28, 2024 16:32
@hamadodene
Copy link
Contributor

@NiccoMlt Reactor Netty 1.1.24 has been released with the fix for the issue we requested: https://github.com/reactor/reactor-netty/releases/tag/v1.1.24
After integrating the fix, could you please check my issue here as well: #506
Perhaps now we can move forward and finalize this PR.

@NiccoMlt
Copy link
Contributor Author

Sure

@NiccoMlt NiccoMlt force-pushed the 410-http2-enable-http2-h2 branch 4 times, most recently from 8041f7a to 51f0c6d Compare December 10, 2024 12:55
@NiccoMlt NiccoMlt force-pushed the 410-http2-enable-http2-h2 branch from db280cb to 2f96899 Compare December 10, 2024 16:25
@NiccoMlt NiccoMlt marked this pull request as ready for review December 10, 2024 17:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants