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.

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

Frontier console node admin interface #485

Merged
merged 17 commits into from
Mar 17, 2015

Conversation

zelig
Copy link
Contributor

@zelig zelig commented Mar 15, 2015

based on #484
This PR implements a node admin interface as speced here
https://github.com/ethereum/go-ethereum/wiki/Frontier-Console
accessible via the admin object in the Frontier JS Console and VM

p2p server and discover/table exposes Node to support nodeInfo()
eth implements NodeInfo() and PeersInfo() to support the node admin interface

see #459

zelig added 7 commits March 15, 2015 13:13
- require became loadScript(), no require is supplied
- bignumber_js.go: heredoc v2.0.3 minified fixed for otto Regexp incompatibility https://github.com/robertkrimen/otto#regular-expression-incompatibility
- bignumber.min.js also updated in mist/assets/ext
- ethereum_js.go: latest master minified
- assetPath in constructor
- Eval/Exec/Handle/ToVal nice API
- jsre tests
- camelcase fields dont play nice with otto value magic: JsonRpc -> Jsonrpc, ID -> Id
- jeth: ethereum.js rpc transport provider direct bridge between js and ethereumApi via otto jsre
- js subcommand for vm
- console for Frontier console interactive REPL
- jspath in cli
- integrate jeth apiBindings
- NodeInfo, PeerInfo types, PeersInfo(), NodeInfo() in eth/backend to support node admin IF
- start net only if MaxPeers > 0, close extraDb in Stop()
- defer s.extraDb.Close()
@obscuren
Copy link
Contributor

Fix the tests please

Caps string
RemoteAddress string
LocalAddress string
}
Copy link
Contributor

Choose a reason for hiding this comment

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

Why is this type necessary? Can't it just return peers directly?
Same for NodeInfo.

@@ -251,6 +309,7 @@ func (s *Ethereum) StateDb() common.Database { return s.stateDb }
func (s *Ethereum) ExtraDb() common.Database { return s.extraDb }
func (s *Ethereum) IsListening() bool { return true } // Always listening
func (s *Ethereum) PeerCount() int { return s.net.PeerCount() }
func (s *Ethereum) PeerInfo() int { return s.net.PeerCount() }
Copy link
Contributor

Choose a reason for hiding this comment

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

What is this about? Is that intended?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Absolutely not, thanks for spotting. Will fix in #512

fjl added a commit that referenced this pull request Mar 17, 2015
Frontier console node admin interface
@fjl fjl merged commit 786a58d into ethereum:develop Mar 17, 2015
@fjl fjl removed the in progress label Mar 17, 2015
@obscuren
Copy link
Contributor

👍

@zelig zelig deleted the frontier/nodeadmin.js branch December 8, 2015 20:48
nolash pushed a commit to nolash/go-ethereum that referenced this pull request May 3, 2018
* swarm/network/stream: fix syncer ticker leak

* swarm/network/stream: a better ticker stop for syncer SetNextBatch
ngtuna added a commit to ngtuna/tomochain that referenced this pull request Aug 16, 2019
…gner-at-checkpoint

Fix ethereum#482: Ignore order of masternodes list
maoueh pushed a commit to streamingfast/go-ethereum that referenced this pull request Nov 12, 2021
…thereum#23635) (ethereum#485)

* core/state/snapshot: fix BAD BLOCK error when snapshot is generating

* core/state/snapshot: alternative fix for the snapshot generator

* add comments and minor update

Co-authored-by: Martin Holst Swende <[email protected]>

* core/state/snapshot: fix BAD BLOCK error when snapshot is generating

* core/state/snapshot: alternative fix for the snapshot generator

* add comments and minor update

Co-authored-by: Martin Holst Swende <[email protected]>

Co-authored-by: Ziyuan Zhong <[email protected]>
Co-authored-by: Martin Holst Swende <[email protected]>
maoueh pushed a commit to streamingfast/go-ethereum that referenced this pull request Jan 6, 2023
* initial commit

* safe latest error

* remove json listener

* fix: debug block unit test

* remove jsonrpc listener from server

* fix: consume network's find port in heimdall client test

* update key in log while startup

* linters

* fix

Co-authored-by: Manav Darji <[email protected]>
s1na pushed a commit to s1na/go-ethereum that referenced this pull request Dec 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants