Connect TCP with an invalid address is silent in the event log
I maintainer di solito rispondono entro 3 giorni
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 77/100
Direzione di ricerca
Inizia con connect_tcp_with_validation_and_save in relay/src/update.rs, poi leggi tcp_validation_text in relay/src/view.rs e validate_tcp_addr in relay/src/state.rs. Segui come gli errori di connessione arrivano al registro eventi e come vengono visualizzati gli errori di convalida. Il lavoro è completato quando gli indirizzi vuoti e malformati rifiutati compaiono in Recent events con messaggi distinti.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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.
- Lingua principale
- Python
- Stelle
- 4
- Fork
- 8
- Merge medio
- 3g 6h
- PR unite (30g)
- 12
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di oss-slu/PilotDataSynchronization
-
area:ml bug python
Difficoltà 2/5 1-3 ore Idoneità per principianti 83/100
oss-slu/PilotDataSynchronization#232 ·
I maintainer di solito rispondono entro 3 giorni
-
area:docs area:xplane-plugin documentation ready-for-human
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
oss-slu/PilotDataSynchronization#194 · 1 commento ·
I maintainer di solito rispondono entro 3 giorni
-
Live performance groups: relay → Mirror → Python sidecarForse già presa @Phlabry l’ha presa 7 giorni fa. Apertaarea:ml area:relay needs-triage
oss-slu/PilotDataSynchronization#245 · 1 commento · 2 assegnatari ·
I maintainer di solito rispondono entro 3 giorni
-
Relay rejects host names, so a changing iMotions IP can't be entered as a nameForse già presa Una pull request collegata a questa issue è aperta o già unita. Apertaarea:relay enhancement needs-triage rust
Difficoltà 3/5 1-2 giorni Idoneità per principianti 48/100
oss-slu/PilotDataSynchronization#240 ·
I maintainer di solito rispondono entro 3 giorni
-
area:ml enhancement needs-info
Difficoltà 5/5 Più di una settimana Idoneità per principianti 1/100
oss-slu/PilotDataSynchronization#238 ·
I maintainer di solito rispondono entro 3 giorni
Tutte le issue di oss-slu/PilotDataSynchronization
Issue simili
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 82/100
btclib-org/btclib-node#2083 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
Gathin23/cleanyourtv#2089 ·
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
jxxghp/MoviePilot#7212 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
I maintainer di solito rispondono entro 1 giorno
-
cli dx rust
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
NekomyaDev/nudge#278 ·
I maintainer di solito rispondono entro 1 giorno