cloudflare/boringtun

Keeping timers relative to start of tunnel initiation is confusing

Offen

#309 geöffnet am 24.08.2022

Auf GitHub ansehenContributor Guide
 (6 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Rust (321 Forks)batch import
code qualitygood first issue

Repository-Metriken

Stars
 (4.849 Sterne)
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
Verstehe die aktuelle Timer Implementierung in boringtun, insbesondere wie `Duration` verwendet wird, um absolute Zeiten darzustellen. Untersuche das Feld `Tunn::timers` und die Funktion `update session timers`. Erwäge eine Refaktorisierung, um `Instant` für absolute Zeitstempel zu verwenden, und stelle sicher, dass alle Vergleiche und Berechnungen entsprechend angepasst werden.
Tech Stack
rust
Domain
backendnetworking
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-3 Stunden
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.
Klar
Voraussetzungen
Rust
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65