ipfs/distributions

Can only publish to an IPFS daemon running on localhost:5001

Open

#345 opened on May 14, 2021

View on GitHub
 (3 comments) (0 reactions) (0 assignees)Less (31 forks)auto 404
P3dif/easyeffort/hourshelp wantedkind/enhancement

Repository metrics

Stars
 (49 stars)
PR merge metrics
 (PR metrics pending)

Description

We create an IPFS HTTP Client here https://github.com/ipfs/distributions/blob/7d79fddb8dee3fda74a5b0728da8478db1b2f97d/scripts/dist.js#L23 which per the examples uses the default local go-ipfs port https://github.com/ipfs/js-ipfs/tree/master/packages/ipfs-http-client#example.

It would be great if this could use the IPFS_PATH environment variable to attempt to locate the API file which will have a multiaddr in it and then pass that multiaddr into the HTTP client.

  1. Read environment variable
  2. Read API addr from file https://github.com/ipfs/go-ipfs-http-client/blob/3af36210f80fb86aae50da582b494ceddd64c3de/api.go#L70
  3. Convert multiaddr into IPv4/IPv6 URL and pass it into the client

It may even be worth upstreaming these few lines of code into the HTTP Client as an alternative construction option (WDYT @achingbrain)

Contributor guide

Research direction
Read the IPFS PATH environment variable, then read the API file from that path to get the multiaddr, convert it to a URL, and pass it to the IPFS HTTP client.
Tech stack
javascriptnodejs
Domain
backendapi
Issue type
Feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
GitBasic JavaScriptUnderstanding of IPFS
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
60