cloudflare/boringtun

Keeping timers relative to start of tunnel initiation is confusing

Open

#309 aberto em 24 de ago. de 2022

Ver no GitHub
 (5 comments) (0 reactions) (0 assignees)Rust (321 forks)batch import
code qualitygood first issue

Métricas do repositório

Stars
 (4.849 stars)
Métricas de merge de PR
 (Mesclagem média 318d 18h) (3 fundiu PRs em 30d)

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.

Guia do colaborador

Direção de pesquisa
A issue propõe reformular o sistema de temporizadores para usar `Instant` em vez de `Duration` para rastrear quando os eventos ocorreram. Especificamente, `update session timers` recebe um parâmetro `Duration` para `time now` que deveria ser um `Instant`. O campo `Tunn::timers` armazena valores `Duration` que representam tempos absolutos; estes devem ser alterados para `Instant`. A mudança afetará como os temporizadores são calculados em relação ao início do túnel. Os arquivos chave a examinar incluem `tunnel.rs` e qualquer módulo que lide com temporizadores. A discussão na issue pode fornecer mais insights sobre as alterações necessárias.
Pilha de tecnologia
rust
Domain
backend
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Rust Duration/Instant typesboringtun codebase structure
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40