cloudflare/boringtun

Keeping timers relative to start of tunnel initiation is confusing

Open

#309 ouverte le 24 août 2022

Voir sur GitHub
 (5 commentaires) (0 réactions) (0 assignés)Rust (321 forks)batch import
code qualitygood first issue

Métriques du dépôt

Stars
 (4 849 stars)
Métriques de merge PR
 (Merge moyen 318j 18h) (3 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Le problème propose de retravailler le système de minuteries pour utiliser `Instant` au lieu de `Duration` pour suivre le moment où les événements se sont produits. Plus précisément, `update session timers` prend un paramètre `Duration` pour `time now` qui devrait être un `Instant`. Le champ `Tunn::timers` stocke des valeurs `Duration` qui représentent des temps absolus ; celles ci doivent être changées en `Instant`. Le changement affectera la façon dont les minuteries sont calculées par rapport au début du tunnel. Les fichiers clés à examiner incluent `tunnel.rs` et tout module gérant les minuteries. La discussion dans le problème peut fournir des informations supplémentaires sur les modifications nécessaires.
Stack technique
rust
Domaine
backend
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Rust Duration/Instant typesboringtun codebase structure
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40