ipfs/ipfs-companion

Check API on both ipv4 and ipfv6

Open

#908 aberto em 8 de jul. de 2020

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)JavaScript (364 forks)batch import
P3effort/daysexp/experthelp wantedneed/analysisneed/maintainers-input

Métricas do repositório

Stars
 (1.992 stars)
Métricas de merge de PR
 (Mesclagem média 53d 3h) (1 fundiu PR em 30d)

Description

Fun edge case that is super painful to debug:

i had a lingering service bound to: [::1]:8080 (ipv6), so the browser was trying that first. go-ipfs does not bind to that for the gateway so i wasn't getting any feedback ipfs companion could also check for both -- as localhost in the browser will try ipv6 first

Needs more analysis, but initial thoughts:

  • enforce use of raw IP for API port (if it does not yet), convert localhost if needed (removes ambiguity which v is used)
  • support entering API URL as ip4 and ip6 (needs fix for #904)
  • when one fails, try another one, and fixup URL if second one is successful (test with /api/v0/id)

Guia do colaborador

Direção de pesquisa
Investigue como a extensão atualmente lida com a resolução de URL da API e implemente a lógica para tentar endereços IPv4 e IPv6.
Pilha de tecnologia
javascript
Domain
frontendapi
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
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.
Principalmente claro
Prerequisites
JavaScriptBrowser extension development
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
55