ipfs/ipfs-companion

Check API on both ipv4 and ipfv6

Open

#908 ouverte le 8 juil. 2020

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)JavaScript (364 forks)batch import
P3effort/daysexp/experthelp wantedneed/analysisneed/maintainers-input

Métriques du dépôt

Stars
 (1 992 stars)
Métriques de merge PR
 (Merge moyen 53j 3h) (1 PR mergée en 30 j)

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)

Guide contributeur

Direction de recherche
Enquêtez sur la façon dont l'extension gère actuellement la résolution d'URL de l'API et implémentez une logique pour essayer à la fois les adresses IPv4 et IPv6.
Stack technique
javascript
Domaine
frontendapi
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
JavaScriptBrowser extension development
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
55