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

WIP: IPFS API (core+http+cli) #65

Merged
merged 3 commits into from
Feb 10, 2016
Merged

WIP: IPFS API (core+http+cli) #65

merged 3 commits into from
Feb 10, 2016

Conversation

daviddias
Copy link
Member

Upgraded the doc to reflect and indicate how we are organising our API specs. Moved from a level org to a system org (calling something level 1 and 2, when inside level 1 there are extensions and optional features made things a tad confusing).

Currently, the IPFS Core API (the programmatic interface) is still kind in flux (and that is ok), so no need to pressure ourselves into locking it down. The IPFS CLI happens as needed and as the system evolve. The only API spec that needs to get more stable, is the HTTP API as a lot of other libraries and applications depend on it.

Currently, the HTTP API is a mix of RPC and RESTful (it really isn't either), which makes it weird to assess. Having a RPC API gives us the chance of layering it over any other transport, while not having a RESTful one makes it weird for Web Apps libraries.

Let's continue the endeavour of documenting the current HTTP API through Apiary (here http://github.com/ipfs/api) so that then we can migrate through PR and reviews

@ghost
Copy link

ghost commented Jan 11, 2016

mix of RPC and RESTful (it really isn't either)

I agree that it's very low in the scale of RESTfulness, why call it RESTful at all though? The explicit goal is to mirror the CLI as much as possible.

@daviddias daviddias mentioned this pull request Jan 11, 2016
18 tasks

# Table of contents

- [%N% Introduction]()
Copy link
Member

Choose a reason for hiding this comment

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

what is %N%?

Copy link
Member Author

Choose a reason for hiding this comment

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

It is just a placeholder for the future number of the chapter


> Official spec of the APIs that ipfs implementations must conform to, as well as test suites to ensure they do.
> This document presentes the specifications for the IPFS APIs, namely: 'core', the programmatic interface, 'http', a networked API interface over HTTP and 'cli', the command line interface to interact with IPFS.
Copy link
Member

Choose a reason for hiding this comment

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

presentes >> presents

@RichardLitt
Copy link
Member

Added one spelling note, otherwise, looks good to me.

daviddias added a commit that referenced this pull request Feb 10, 2016
WIP: IPFS API (core+http+cli)
@daviddias daviddias merged commit 1a17d32 into master Feb 10, 2016
@daviddias daviddias deleted the ipfs/api branch February 10, 2016 07:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants