AdguardTeam/AdGuardHome

Rewrite networking in package dhcpd to only use gopacket

Open

#4.923 aberto em 15 de set. de 2022

Ver no GitHub
 (0 comments) (1 reaction) (1 assignee)Go (2.333 forks)batch import
P3: Mediumenhancementexternal libshelp wantedresearch

Métricas do repositório

Stars
 (34.000 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

Prerequisites

  • I have checked the Wiki and Discussions and found no answer

  • I have searched other issues and found no duplicates

  • I want to request a feature or enhancement and not ask a question

Description

What problem are you trying to solve?

Package dhcpd currently uses a lot of different modules and packages for networking, including github.com/google/gopacket, github.com/insomniacslk/dhcp, github.com/mdlayher/ethernet, and the deprecated github.com/mdlayher/raw. This creates all sorts of issues, including but not limited to:

  1. portability;
  2. API compatibility;
  3. difficulty of package update and maintenance.

Proposed solution

Module github.com/google/gopacket seems to have either all or most of the things we require. If I am not mistaken, we could even enable DHCP on Windows, if we make sure that the networking works well there.

Alternatives considered

Go on as we do. See the problems with this approach above.

Guia do colaborador

Direção de pesquisa
Examine o código de rede atual no pacote 'dhcpd' para identificar todos os usos de 'github.com/insomniacslk/dhcp', 'github.com/mdlayher/ethernet' e 'github.com/mdlayher/raw'. Em seguida, pesquise como substituir cada um por equivalentes do 'github.com/google/gopacket'. Preste atenção especial à compatibilidade com o Windows, conforme mencionado na issue. Como a issue está atribuída, pode valer a pena verificar o progresso do responsável ou discutir com os mantenedores para evitar duplicação.
Pilha de tecnologia
go
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.
Mais de 1 semana
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
Gonetworking
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
25