mumble-voip/mumble

Connection Dialog Should Display Ping Issues (timeouts, packetloss)

Offen

#3.158 geöffnet am 09.07.2017

Auf GitHub ansehen
 (4 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (1.062 Forks)batch import
clientfeature-requesthelp wantedui

Repository-Metriken

Stars
 (5.642 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

I had internet connection issues the last two days and noticed the server stats are only updated on successful pings. That means that if you had 1 successful 30ms ping with user count and then none for 10 minutes the displayed info is 10 minutes old and wrong (/does not indicate connection issues).

If we start with timeouts we do not display anything. Maybe some negative feedback would be useful then as well (use case: user has only 1 favourite server; does not see that we ping unsuccessfully, connect button is greyed out).

Contributor Guide

Research-Richtung
Das Problem fordert, dass der Verbindungsdialog Indikatoren für Ping Fehler (Timeouts, Paketverlust) anzeigt und sich nicht nur auf erfolgreiche Pings verlässt. Um dies zu implementieren, müsste die Ping Verarbeitungslogik im Mumble Client geändert werden, wahrscheinlich in den Dateien, die mit dem Verbindungsdialog zusammenhängen (z. B. `src/mumble/Connection.cpp` oder ähnliches). Das derzeitige Verhalten aktualisiert Statistiken nur bei erfolgreichen Pings, daher beinhaltet die Lösung die Erkennung und Anzeige fehlgeschlagener Pings, möglicherweise durch Anzeige von 'Verbindung verloren' oder eines Latenzdiagramms mit fehlenden Datenpunkten.
Tech Stack
cpp
Domain
frontendbackend
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
C++Qt
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60