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

RPC method to get list of methods/notifications #437

Closed
sonnyp opened this issue Jul 29, 2015 · 8 comments
Closed

RPC method to get list of methods/notifications #437

sonnyp opened this issue Jul 29, 2015 · 8 comments

Comments

@sonnyp
Copy link
Contributor

sonnyp commented Jul 29, 2015

Would be awesome to have a new RPC method to get list of RPC methods/notifications.

This would help RPC clients who do not have the possibility to catch functions call and have to manually declare a function for each aria2 method/notification for example: https://github.com/sonnyp/aria2.js/blob/master/src/index.js#L243

rpc. namespace could be use for this http://www.jsonrpc.org/specification#extensions
@sonnyp
Copy link
Contributor Author

sonnyp commented Dec 26, 2015

additionally it would be very helpful to return a description of the method and its signature (accepted arguments and types), maybe using JSON schema ?

@sonnyp sonnyp mentioned this issue Dec 26, 2015
Merged
tatsuhiro-t added a commit that referenced this issue Dec 26, 2015
@tatsuhiro-t
Add system.listMethods RPC method
d0ccb39 
system.listMethods just returns the available RPC method names.  It
can be executed without secret token, because it just returns the
method names, and does not alter anything.

See GH-437
@tatsuhiro-t
Copy link
Collaborator

d0ccb39 adds system.listMethods which just returns the available RPC method names.
This is an existing extension for XML-RPC which is supported by many RPC server implementations.

@sonnyp
Copy link
Contributor Author

sonnyp commented Dec 26, 2015

@tatsuhiro-t thanks, that's useful

@sonnyp
Copy link
Contributor Author

sonnyp commented Dec 27, 2015

aria2 rpc cli https://github.com/sonnyp/aria2.js/blob/master/bin/README.md if it's of any help to you / the project

@sonnyp
Copy link
Contributor Author

sonnyp commented Dec 27, 2015

the methods description/signature is an other story, will open a new issue if needs be.

@sonnyp sonnyp closed this as completed Dec 27, 2015
@sonnyp sonnyp mentioned this issue Dec 30, 2015
Closed
@sonnyp
Copy link
Contributor Author

sonnyp commented Dec 30, 2015

@tatsuhiro-t could you make a new release including the listMethods method?

@tatsuhiro-t
Copy link
Collaborator

We'll do the next release in the next month.

@sonnyp
Copy link
Contributor Author

sonnyp commented Dec 30, 2015

👍

@sonnyp sonnyp mentioned this issue Apr 8, 2016
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@sonnyp @tatsuhiro-t