-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
Add readonly api to gateway #1581
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
Conversation
cc @harlantwood as he'll be interested to follow this |
The remaining todo here is to write test cases. Suggestions? |
Great to see this moving! We're doing interesting work on the graph visualization side... Will share more soon. |
5823cde
to
3357d2f
Compare
@jbenet are the test cases adequate? If so, RFM. Otherwise testing all the API commands might be redundant. |
Based on ipfs#1389 License: MIT Signed-off-by: rht <[email protected]>
3357d2f
to
dd99a70
Compare
@rht looks great to me! so simple :) 👍 |
@harlantwood @krl read only api is in! 👍 thanks @rht |
Awesome thanks @rht! |
@jbenet I'm interested to know when this is released to the public gateway. Is there a way we track what is released there? Otherwise maybe comment here when it's live. |
@lgierth could you deploy to public gateways? — On Sat, Aug 15, 2015 at 6:32 PM, Harlan T Wood [email protected]
|
👍 |
}, | ||
}, | ||
"cat": CatCmd, | ||
"commands": CommandsDaemonCmd, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wait, one more small fix: this lists the Root commands instead of RootRO.
API is sanitized here #1583 |
Based on #1389
(wip)