-
Notifications
You must be signed in to change notification settings - Fork 0
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
Jetty core Handler that wraps Spring HttpHandler as an adaptor #1
Conversation
@gregw you should be able to assign to me now |
Handler
that wraps Spring HttpHandler
as an adaptor
Handler
that wraps Spring HttpHandler
as an adaptor
@lachlan-roberts see jetty/jetty.project#11288, which contains a fix to the |
@gregw I have pushed the commits to implement the write side. |
@lachlan-roberts what was the conclusion about websocket upgrade? |
I've opened draft spring-projects#32097 to solicit more feedback |
@lachlan-roberts unsurprisingly, we are failing websocket tests: org.springframework.web.reactive.socket.WebSocketIntegrationTests |
WebSocket tests are passing now. There is a |
…pResponse Signed-off-by: Lachlan Roberts <lachlan@webtide.com>
Jetty has an unset response status of 0, this should be converted to 200 for spring.
When mutating the request, a mutable HttpFields instance is created.
Spring appears to want the unadulterated raw path complete with parameters etc.
Signed-off-by: Lachlan Roberts <lachlan@webtide.com>
Signed-off-by: Lachlan Roberts <lachlan@webtide.com>
…cookies Signed-off-by: Lachlan Roberts <lachlan@webtide.com>
Signed-off-by: Lachlan Roberts <lachlan@webtide.com>
Signed-off-by: Lachlan Roberts <lachlan@webtide.com>
Signed-off-by: Lachlan Roberts <lachlan@webtide.com>
Signed-off-by: Olivier Lamy <olamy@apache.org>
53af169
to
bbd2573
Compare
Closed in favor of spring-projects#32097 |
Implement a Jetty core
Handler
that uses a springHttpHandler
without the overhead of servlets and using reactive async abstraction.