Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

jsonrpc-apis, ws-apis and ipc-apis update #8234

Merged
merged 4 commits into from
Mar 28, 2018
Merged

Conversation

Tbaut
Copy link
Contributor

@Tbaut Tbaut commented Mar 27, 2018

No description provided.

@Tbaut Tbaut requested a review from tomusdrw March 27, 2018 13:33
@parity-cla-bot
Copy link

It looks like @Tbaut signed our Contributor License Agreement. 👍

Many thanks,

Parity Technologies CLA Bot

Copy link
Collaborator

@tomusdrw tomusdrw left a comment

Choose a reason for hiding this comment

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

Tiny inconsistency in safe definition.

@@ -452,7 +452,7 @@ usage! {

ARG arg_jsonrpc_apis: (String) = "web3,eth,pubsub,net,parity,parity_pubsub,traces,rpc,shh,shh_pubsub", or |c: &Config| c.rpc.as_ref()?.apis.as_ref().map(|vec| vec.join(",")),
"--jsonrpc-apis=[APIS]",
"Specify the APIs available through the JSONRPC interface. APIS is a comma-delimited list of API name. Possible name are all, safe, web3, eth, net, personal, parity, parity_set, traces, rpc, parity_accounts, pubsub, parity_pubsub, shh, shh_pubsub, signer, secretstore. You can also disable a specific API by putting '-' in the front: all,-personal.",
"Specify the APIs available through the JSONRPC interface. APIS is a comma-delimited list of API name. Possible names are: all, safe, web3, net, eth, pubsub, personal, signer, parity, parity_pubsub, parity_accounts, parity_set, traces, rpc, secretstore, shh, shh_pubsub. You can also disable a specific API by putting '-' in the front, example: all,-personal. safe contains following apis: traces, parity_pubsub",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Actually safe contains all APIs from public list:
https://github.com/paritytech/parity/blob/2e09db185f4cd443a4753c79458ce1105f60fcea/parity/rpc_apis.rs#L623-L632

plus traces and ParityPubSub.

@Tbaut
Copy link
Contributor Author

Tbaut commented Mar 27, 2018

Thanks @tomusdrw for spotting this :)

@Tbaut Tbaut added A2-insubstantial 👶 Pull request requires no code review (e.g., a sub-repository hash update). M1-ci 🙉 Continuous integration. labels Mar 27, 2018
@Tbaut Tbaut added this to the 1.11 milestone Mar 27, 2018
@openethereum openethereum deleted a comment from parity-cla-bot Mar 27, 2018
@@ -452,7 +452,7 @@ usage! {

ARG arg_jsonrpc_apis: (String) = "web3,eth,pubsub,net,parity,parity_pubsub,traces,rpc,shh,shh_pubsub", or |c: &Config| c.rpc.as_ref()?.apis.as_ref().map(|vec| vec.join(",")),
"--jsonrpc-apis=[APIS]",
"Specify the APIs available through the JSONRPC interface. APIS is a comma-delimited list of API name. Possible name are all, safe, web3, eth, net, personal, parity, parity_set, traces, rpc, parity_accounts, pubsub, parity_pubsub, shh, shh_pubsub, signer, secretstore. You can also disable a specific API by putting '-' in the front: all,-personal.",
"Specify the APIs available through the JSONRPC interface. APIS is a comma-delimited list of API name. Possible names are: all, safe, web3, net, eth, pubsub, personal, signer, parity, parity_pubsub, parity_accounts, parity_set, traces, rpc, secretstore, shh, shh_pubsub. You can also disable a specific API by putting '-' in the front, example: all,-personal. safe contains following apis: web3, net, eth, pubsub, parity, parity_pubsub, traces, rpc, shh, shh_pubsub",
Copy link
Collaborator

@niklasad1 niklasad1 Mar 27, 2018

Choose a reason for hiding this comment

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

Specify the APIs available through the JSONRPC interface which is a comma-delimited list of API names.

Copy link
Collaborator

Choose a reason for hiding this comment

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

If you agree change at the other places as well otherwise ignore it :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Agree

@Tbaut Tbaut merged commit 2632310 into master Mar 28, 2018
@5chdn 5chdn deleted the Minor-cli-description-update branch March 28, 2018 09:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A2-insubstantial 👶 Pull request requires no code review (e.g., a sub-repository hash update). M1-ci 🙉 Continuous integration.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants