Skip to content

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

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

Merged
merged 2 commits into from
Mar 23, 2021

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 b5c6@protonmail.com

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 <b5c6@protonmail.com>
@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 <b5c6@protonmail.com>
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 <b5c6@protonmail.com>

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

Date: 2021-03-16 06:13:23-05:00
Signed-off-by: meows <b5c6@protonmail.com>
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