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.

Sign up for GitHub

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

Jump to bottom

Files API should handle ipfs protocol paths #2837

Open
Kubuxu opened this issue Jun 11, 2016 · 2 comments
Open

Files API should handle ipfs protocol paths #2837

Kubuxu opened this issue Jun 11, 2016 · 2 comments
Labels
exp/novice Someone with a little familiarity can pick up help wanted Seeking public contribution on this issue kind/enhancement A net-new feature or improvement to an existing feature status/deferred Conscious decision to pause or backlog

Comments

@Kubuxu
Copy link
Member

Kubuxu commented Jun 11, 2016 •
edited by whyrusleeping
Loading

As files API provides functions not available elsewhere (ipfs fiels stat) it would be great if it was handling /ipfs/Qm...AAA paths also.

They are already not fully functional (you can't for example copy files from /ipfs/ directory in files API).

This extends to: /ipns/ and other if they are ever created.
@Kubuxu Kubuxu added kind/enhancement A net-new feature or improvement to an existing feature help wanted Seeking public contribution on this issue exp/novice Someone with a little familiarity can pick up kind/support A question or request for support and removed help wanted Seeking public contribution on this issue labels Jun 11, 2016
@whyrusleeping
Copy link
Member

I would not be opposed to this, lets do it.

@whyrusleeping whyrusleeping added the help wanted Seeking public contribution on this issue label Jun 13, 2016
@Kubuxu Kubuxu removed the kind/support A question or request for support label Jun 13, 2016
@Kubuxu Kubuxu changed the title Files API should handle /ipfs/ paths Files API should handle ipfs protocol paths Jun 13, 2016
@whyrusleeping whyrusleeping mentioned this issue Jun 13, 2016
Closed
@Kubuxu Kubuxu added this to the Ipfs 0.4.3 milestone Jun 22, 2016
@whyrusleeping
Copy link
Member

This isnt as simple as I had initially thought. Its going to require either some moderate refactoring, or lots of special casing (or both). I'm going to bump it from the 0.4.3 milestone

@whyrusleeping whyrusleeping removed this from the ipfs-0.4.3 milestone Jul 6, 2016
@dokterbob dokterbob mentioned this issue Aug 13, 2016
Closed
4 tasks
@whyrusleeping whyrusleeping added the status/deferred Conscious decision to pause or backlog label Nov 28, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
exp/novice Someone with a little familiarity can pick up help wanted Seeking public contribution on this issue kind/enhancement A net-new feature or improvement to an existing feature status/deferred Conscious decision to pause or backlog
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@whyrusleeping @Kubuxu