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

WebExtension makes **go-ipfs** full ipfs core api (interface-core-api) available in the browser #41

Closed
flyingzumwalt opened this issue Feb 15, 2017 · 5 comments

Comments

@flyingzumwalt
Copy link
Contributor

flyingzumwalt commented Feb 15, 2017

depends on #11

Web extension makes go-ipfs protocol (interface-core-api) available in the browser through window.ipfs

  • uses js-ipfs-api to talk with the node
  • using the core api vs relying on the gateway should be disabled/enabled by toggling a "Redirect-URLs" option true or false
  • disable by default
@haadcode
Copy link
Member

Started looking into this with the idea that js-ipfs-api is packaged with the extension and exposed to window.ipfs.

What I take from this is that we can't make js-ipfs-api work from the webextension using http://localhost:5001 to communicate with the go-ipfs node like we normally do. Please correct me if I'm wrong.

Next: I'll try other ways to include it.

@victorb
Copy link
Member

victorb commented Feb 21, 2017

@haadcode I think @whyrusleeping started with using what we discussed in the last call, Native Messaging via stdin/stdout. We would have a layer in between the daemon and the extension that would pass json of some sort via stdin/stdout. https://github.com/whyrusleeping/ipfs-webext

@haadcode
Copy link
Member

We might be able to apply the same structure here to expose js-ipfs-api as @victorbjelkholm did in #42 (comment). We'd need that layer in-between, perhaps we could even replace the fetch request in js-ipfs-api with the stdin/out request.

@flyingzumwalt
Copy link
Contributor Author

@haadcode @victorbjelkholm have we made any progress on this? will we be able to close it out tomorrow and demo on Monday?

@ghost ghost added status/ready Ready to be worked and removed status/in-progress In progress labels Mar 28, 2017
@ghost ghost closed this as completed Apr 18, 2017
@ghost ghost removed the status/ready Ready to be worked label Apr 18, 2017
@lidel
Copy link
Member

lidel commented Mar 6, 2018

Delivered in ipfs/ipfs-companion#330 👍

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

No branches or pull requests

4 participants