Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: use ipfs-provider #113

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

fsdiogo
Copy link
Contributor

@fsdiogo fsdiogo commented Jun 14, 2019

  • Removes js-ipfs-http-client in favour of ipfs-provider
  • Adds an option that checks if a node is running on the default port
    • If there is, connects to it
    • If there isn't, the previous behaviour happens (spawns an in-process node)
    • To disable this check just pass --ipfs-disable-providers, it's enabled by default

@autonome
Copy link

@achingbrain Can this merge?

@@ -42,7 +42,7 @@
"express": "^4.16.4",
"express-http-proxy": "^1.5.1",
"ipfs": "^0.36.3",
"ipfs-http-client": "^32.0.1",
"ipfs-provider": "^0.2.1",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool module!

Copy link
Member

@daviddias daviddias Sep 14, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will pick up if I'm running IPFS Desktop, correct? 👌🏽

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Correct 👍

@@ -85,6 +85,8 @@ Options:
timing out [default: 5000]
--ipfs-mfs-prefix Which mfs prefix to use
[default: "/npm-registry"]
--ipfs-disable-providers Wether to disable the search for running nodes

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo: Wether -> Whether

@diaakasem
Copy link

When do you think you can merge this pull-request ?

@resynth1943
Copy link

/cc @achingbrain
Also, how does one request to be a maintainer of this project?

@Arlodotexe
Copy link

Bump

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants