ipfs/ipfs-companion

Check API on both ipv4 and ipfv6

Open

#908 aperta il 8 lug 2020

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)JavaScript (364 fork)batch import
P3effort/daysexp/experthelp wantedneed/analysisneed/maintainers-input

Metriche repository

Star
 (1992 star)
Metriche merge PR
 (Merge medio 53g 3h) (1 PR mergiata in 30 g)

Descrizione

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)

Guida contributor

Direzione di ricerca
Indaga come l'estensione gestisce attualmente la risoluzione dell'URL dell'API e implementa la logica per provare sia gli indirizzi IPv4 che IPv6.
Tech stack
javascript
Dominio
frontendapi
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
JavaScriptBrowser extension development
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
55