Simplify VMConnect RDCleanPath handling
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 48/100
- Tipo de issue
- Refactorización
- Claridad
- Bastante claro
- Estado de actividad
- Tranquilo
- Stack tecnológico
- rust
- Área
- networking
Línea de trabajo
Empieza en rd_clean_path.rs y lee las pruebas existentes de la estructura de la solicitud, junto con la prueba de regresión del formato de cable UTF-16 non-BMP. Ejecuta las pruebas de Rust pertinentes antes y después de la refactorización; se considera terminado cuando los casos están organizados en tablas, el comportamiento de timeout y flush se ha simplificado según lo especificado, y el comportamiento habitual de los protocolos X.224-first y VMConnect permanece sin cambios.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Follow-up to Benoit's review on #1372.
The VMConnect path works, but rd_clean_path.rs carries more compatibility code and branching than it needs.
Simplify it without changing the protocol behavior:
- Match once on
(x224_connection_pdu, preconnection_blob)and classify the request beside its handling. - Remove the redundant X.224 check from the credential-injection path.
- Keep the 10-second timeout only around the VMConnect PCB write.
- Remove the two front-sequence
flush()calls and the buffering comment. - Build one response by mapping the optional X.224 response field.
- Merge the request-shape cases into a table-driven test.
- Keep the non-BMP UTF-16 wire-format regression test.
Once Gateway can consume a compatible IronRDP release exposing RDCleanPathMessage and the fixed PreconnectionBlob encoder, replace the local classifier, response builder, and PCB encoder with those APIs.
The ordinary path must stay X.224-first.
The VMConnect path must stay PCB -> TLS on Gateway, followed by CredSSP -> X.224 on the client.
[!NOTE]
Human-tuned, LLM-assisted content.
- Lenguaje dominante
- Rust
- Estrellas
- 161
- Forks
- 30
- Merge medio
- 2 d 9 h
- PR fusionados (30 d)
- 37
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
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 Devolutions/devolutions-gateway
-
fix(webapp): RDP web form ignores port in "host:port" (non-default RDP port fails)Posiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
Devolutions/devolutions-gateway#1993 ·
Los mantenedores suelen responder en 1 día
-
allow us to disable tls 1.1, 1.2Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
Devolutions/devolutions-gateway#2047 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
Devolutions/devolutions-gateway#1973 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 3-5 días Aptitud para principiantes 55/100
Devolutions/devolutions-gateway#1788 ·
Los mantenedores suelen responder en 1 día
-
Domain Account login issueAbierto
Dificultad 4/5 3-5 días Aptitud para principiantes 20/100
Devolutions/devolutions-gateway#1328 · 1 comentario ·
Los mantenedores suelen responder en 1 día
Todos los issues de Devolutions/devolutions-gateway
Issues similares
-
documentation enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
adorsys/status-list-server#619 ·
Los mantenedores suelen responder en 2 días
-
batch-backport only backports the first 30 matching PRsPosiblemente ocupada @DvirDukhan la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 5 días
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
equinor/septic-config-generator#481 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día