cloudflare/boringtun

Keeping timers relative to start of tunnel initiation is confusing

Open

#309 geöffnet am 24. Aug. 2022

Auf GitHub ansehen
 (5 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Rust (321 Forks)batch import
code qualitygood first issue

Repository-Metriken

Stars
 (4.849 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 318T 18h) (3 gemergte PRs in 30 T)

Beschreibung

Keeping timers this way means that for example update_session_timers takes a time_now param that's a Duration instead of an Instant. All entries in Tunn::timers are Durations when many of them are "the time that something happened" and idiomatically I think these would be better expressed as Instants.

Contributor Guide

Research-Richtung
Das Issue schlägt vor, das Timer System zu überarbeiten, um `Instant` anstelle von `Duration` für die Verfolgung des Zeitpunkts von Ereignissen zu verwenden. Insbesondere nimmt `update session timers` einen `Duration` Parameter für `time now` entgegen, der ein `Instant` sein sollte. Das Feld `Tunn::timers` speichert `Duration` Werte, die absolute Zeiten darstellen; diese sollten in `Instant` geändert werden. Die Änderung wird beeinflussen, wie Timer relativ zum Tunnelstart berechnet werden. Zu prüfende Schlüsseldateien umfassen `tunnel.rs` und alle Module, die Timer verarbeiten. Die Diskussion im Issue könnte weitere Einblicke in die erforderlichen Änderungen liefern.
Tech Stack
rust
Domain
backend
Issue Type
Refactoring
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.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Rust Duration/Instant typesboringtun codebase structure
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40