Connect TCP with an invalid address is silent in the event log
Los mantenedores suelen responder en 4 días
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 77/100
Línea de trabajo
Empieza con connect_tcp_with_validation_and_save en relay/src/update.rs y luego lee tcp_validation_text en relay/src/view.rs y validate_tcp_addr en relay/src/state.rs. Sigue cómo llegan los fallos de conexión al registro de eventos y cómo se muestran los errores de validación. El trabajo está terminado cuando las direcciones vacías y mal formadas rechazadas aparecen en Recent events con mensajes distintos.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Describe the bug
Pressing Connect TCP with an empty or malformed address does nothing visible in the event log. No connection is attempted and no entry is recorded, so the GUI looks like it ignored the press.
connect_tcp_with_validation_and_save (relay/src/update.rs:13) validates first. On failure it sets tcp_addr_validation_error and returns without calling tcp_connect, so none of the paths that normally report a connection problem are reached. The only feedback is one line of plain text rendered by tcp_validation_text (relay/src/view.rs:192), between the address box and the buttons, which is easy to miss.
The event log panel added in #183 exists so connection failures are visible. This is the one connection failure that never reaches it.
To Reproduce
- Start the relay.
- Leave the address box untouched. The greyed
127.0.0.1:9999is placeholder text, not a value, so the field is empty. - Press
Connect TCP. - Nothing appears in Recent events. The mock server never reports a connection. Pressing
Check TCP Connection Statusdoes nothing, because there is no connection to check.
Expected behavior
A rejected address is recorded in the event log like every other connection failure, and the message distinguishes an empty field from a malformed one.
Additional context
The empty-field case is the one that costs time, because validate_tcp_addr (relay/src/state.rs:345) returns the same "Invalid IP address" text for an empty string as for a typo, which reads as though something was typed wrong rather than not typed at all.
Hit while testing #234: roughly half an hour went to diagnosing a relay that appeared not to connect, with nothing in the log to work from.
- Lenguaje dominante
- Rust
- Estrellas
- 4
- Forks
- 9
- Merge medio
- 5 d 13 h
- PR fusionados (30 d)
- 13
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una 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 oss-slu/PilotDataSynchronization
-
cluster_pilots.py shows a traceback for a bad --k and leaves outputs when --truth is invalidAbiertoarea:ml bug python
Dificultad 2/5 1-3 horas Aptitud para principiantes 83/100
oss-slu/PilotDataSynchronization#232 ·
Los mantenedores suelen responder en 4 días
-
area:docs area:xplane-plugin documentation ready-for-human
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
oss-slu/PilotDataSynchronization#194 · 1 comentario ·
Los mantenedores suelen responder en 4 días
-
No CI builds or tests the Rust cratesPosiblemente ocupada @PRINCEKK122 la tomó hace 1 día. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 25/100
oss-slu/PilotDataSynchronization#252 ·
Los mantenedores suelen responder en 4 días
-
Dificultad 3/5 1-2 días Aptitud para principiantes 63/100
oss-slu/PilotDataSynchronization#251 ·
Los mantenedores suelen responder en 4 días
-
Relay rejects host names, so a changing iMotions IP can't be entered as a namePosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abiertoarea:relay enhancement needs-triage rust
Dificultad 3/5 1-2 días Aptitud para principiantes 48/100
oss-slu/PilotDataSynchronization#240 ·
Los mantenedores suelen responder en 4 días
Todos los issues de oss-slu/PilotDataSynchronization
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
Los mantenedores suelen responder en 1 día
-
compiler P4 perf
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
HerringtonDarkholme/hd-lang#1044 ·
Los mantenedores suelen responder en 1 día
-
good first issue help wanted r-docs
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
servo/rust-smallvec#824 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
akitaonrails/ai-usagebar#552 ·
Los mantenedores suelen responder en 1 día