cloudflare/boringtun

Keeping timers relative to start of tunnel initiation is confusing

Ouverte

#309 ouverte le 24 août 2022

Voir sur GitHubGuide contributeur
 (6 commentaires) (0 réaction) (0 personne assignée)Rust (321 forks)batch import
code qualitygood first issue

Métriques du dépôt

Stars
 (4 849 étoiles)
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
Comprenez l'implémentation actuelle des temporisateurs dans boringtun, en vous concentrant sur la façon dont `Duration` est utilisé pour représenter des temps absolus. Examinez le champ `Tunn::timers` et la fonction `update session timers`. Envisagez de refactoriser pour utiliser `Instant` pour les horodatages absolus, en veillant à ce que toutes les comparaisons et opérations arithmétiques soient ajustées en conséquence.
Stack technique
rust
Domaine
backendnetworking
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-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Rust
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65