cloudflare/boringtun

Keeping timers relative to start of tunnel initiation is confusing

Aberta

#309 aberto em 24 de ago. de 2022

Ver no GitHubGuia do colaborador
 (6 comentários) (0 reação) (0 responsável)Rust (321 forks)batch import
code qualitygood first issue

Métricas do repositório

Stars
 (4.849 estrelas)
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
Entenda a implementação atual dos temporizadores no boringtun, focando em como `Duration` é usado para representar tempos absolutos. Investigue o campo `Tunn::timers` e a função `update session timers`. Considere refatorar para usar `Instant` para timestamps absolutos, garantindo que todas as comparações e aritmética sejam ajustadas adequadamente.
Pilha de tecnologia
rust
Domain
backendnetworking
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-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Rust
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65