Skip to content

Conversation

@meowsbits
Copy link
Contributor

@meowsbits meowsbits commented Mar 8, 2021

Corresponding CLI flags --rpc have been moved to --http.

This moves the admin module HTTP RPC start/stop
methods to an equivalent namespace.

This notes admin.StartRPC and admin.StopRPC as deprecated (and logs a WARN if called) but does not remove them.

Rel #22263

Date: 2021-03-08 08:00:11-06:00
Signed-off-by: meows [email protected]

Corresponding CLI flags --rpc have been moved to --http.

This moves the admin module HTTP RPC start/stop
methods to an equivalent namespace.

Rel ethereum#22263

Date: 2021-03-08 08:00:11-06:00
Signed-off-by: meows <[email protected]>
@meowsbits meowsbits requested review from fjl and renaynay as code owners March 8, 2021 14:47
@renaynay
Copy link
Contributor

renaynay commented Mar 9, 2021

I think this is a nice change cc @fjl although it would be kind of annoying to offer 2 methods do a single task (while waiting to deprecate startRPC/stopRPC)

Date: 2021-03-16 06:13:23-05:00
Signed-off-by: meows <[email protected]>
Copy link
Member

@karalabe karalabe left a comment

Choose a reason for hiding this comment

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

SGTM

@karalabe karalabe added this to the 1.10.2 milestone Mar 23, 2021
@karalabe karalabe merged commit e862cbf into ethereum:master Mar 23, 2021
atif-konasl pushed a commit to frozeman/pandora-execution-engine that referenced this pull request Oct 15, 2021
…thereum#22461)

* internal/web3ext,node: migrate node admin API (Start|Stop)RPC->HTTP

Corresponding CLI flags --rpc have been moved to --http.

This moves the admin module HTTP RPC start/stop
methods to an equivalent namespace.

Rel ethereum#22263

Date: 2021-03-08 08:00:11-06:00
Signed-off-by: meows <[email protected]>

* internal/web3ext: fix startRPC/HTTP param count (4->5)

Date: 2021-03-16 06:13:23-05:00
Signed-off-by: meows <[email protected]>
JukLee0ira added a commit to JukLee0ira/XDPoSChain that referenced this pull request Jun 23, 2025
gzliudan pushed a commit to XinFinOrg/XDPoSChain that referenced this pull request Jun 23, 2025
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