didn't respect origin server Sec-WebSocket-Extension header
@vietj ya está trabajando en esto.
Desde el 17/2/2023.
Evaluación
Este issue todavía no se ha evaluado.
Descripción
Questions
Header Sec-WebSocket-Extension is used to negotiate the list of supported extensions between WebSocket client and server.
In Websocket response 101 Switch Protocols, even origin server returns without permessage-deflate extension, vertx-http-server proxies websocket message to orign server permessage-deflated.
Undoubly, it will not be functional as orgin server doesn't not support that extension.
Version
Which version(s) did you encounter this bug ?
4.3.5
Context
Do you have a reproducer?
Steps to reproduce
- create a new proxy project with below options:
val proxyClient: HttpClient = vertx.createHttpClient()
val proxyOptions = ProxyOptions()
proxyOptions.supportWebSocket = true
val proxy = HttpProxy.reverseProxy(proxyOptions, proxyClient)
proxy.origin(30000, "localhost")
val options = HttpServerOptions()
options.addWebSocketSubProtocol("tty")
vertx
.createHttpServer(options)
.requestHandler(proxy)
.listen(8888) { http ->
if (http.succeeded()) {
startPromise.complete()
println("HTTP server started on port 8888")
} else {
startPromise.fail(http.cause());
}
}
-
downlown and execute ttyd
ttyd is an open source project to serve terminal in Web interface.
Download from https://github.com/tsl0922/ttyd/releases and run with command./ttyd -p 30000 -b /ttyd bash -
Open http://localhost:8888/ttyd/
Extra
As a workaround, I turn off message compression:
options.setPerMessageWebSocketCompressionSupported(false)
- Anything that can be relevant such as OS version, JVM version
- Lenguaje dominante
- Java
- Estrellas
- 68
- Forks
- 39
- Merge medio
- 2 d 43 min
- PR fusionados (30 d)
- 5
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de eclipse-vertx/vertx-http-proxy
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
-
enhancement
Dificultad 4/5 3-5 días Aptitud para principiantes 56/100
-
bug
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
-
ExceptionHandler for send requests and proxy responses in `HttpProxy`/`ReverseProxy`Quizá libre de nuevo @tsegismont la tomó hace 575 días y no hay ningún pull request abierto. Abiertoenhancement
eclipse-vertx/vertx-http-proxy#114 · 2 comentarios · 1 asignado ·
-
enhancement
Dificultad 4/5 3-5 días Aptitud para principiantes 30/100
eclipse-vertx/vertx-http-proxy#109 · 5 comentarios ·
Todos los issues de eclipse-vertx/vertx-http-proxy
Issues similares
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 68/100
quarkusio/quarkus-agent-mcp#296 ·
Los mantenedores suelen responder en 1 día
-
enhancement web-ui
Dificultad 2/5 1-3 horas Aptitud para principiantes 67/100
jenkinsci/jenkins#27583 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
apache/rocketmq-dashboard#7302 ·
Los mantenedores suelen responder en 3 días
-
area: android good first issue help wanted testing
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Gautam-aman/ResQNet#56 ·
Los mantenedores suelen responder en 1 día