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

Splitting up responsibilities of the JsonRpcMethodsFactory #158

Closed
wants to merge 22 commits into from
Closed

Splitting up responsibilities of the JsonRpcMethodsFactory #158

wants to merge 22 commits into from

Conversation

CjHare
Copy link
Contributor

@CjHare CjHare commented Nov 5, 2019

PR description

  1. Having only one class with the name JsonRpcMethodsFactory (currently there is an interface and a factory, unrelated to each other)
  2. Splitting up the JsonRpcMethodsFactory to follow the Single Responsibility Principle, creating more seller chunks of code.

Fixed Issue(s)

Christopher Hare and others added 22 commits November 4, 2019 17:39
Signed-off-by: Christopher Hare <chris.hare@consensys.net>
Signed-off-by: Christopher Hare <chris.hare@consensys.net>
Signed-off-by: Christopher Hare <chris.hare@consensys.net>
Signed-off-by: Christopher Hare <chris.hare@consensys.net>
Signed-off-by: Christopher Hare <chris.hare@consensys.net>

Signed-off-by: Christopher Hare <chris.hare@consensys.net>
When setting the p2p port to zero and turning on UPNP nat an attempt is
made to map port zero.  This should actually map the opened port
instead.

The core logic is also now set up to throw an exception if a zero local
port is requested.

Signed-off-by: Danno Ferrin <danno.ferrin@gmail.com>
Signed-off-by: Christopher Hare <chris.hare@consensys.net>
Istanbul Mainnet block is 9,069,000.  Update mainnet config.

Signed-off-by: Danno Ferrin <danno.ferrin@gmail.com>
Signed-off-by: Christopher Hare <chris.hare@consensys.net>
The IbftQueryService has been added to the Plugin-api, along with an implementation.

This allows IBFT specific aspects of a block header to be queried (round number, and signers in block).

Signed-off-by: Trent Mohay <trent.mohay@consensys.net>
Signed-off-by: Christopher Hare <chris.hare@consensys.net>
Signed-off-by: Sally MacFarlane <sally.macfarlane@consensys.net>
Signed-off-by: Christopher Hare <chris.hare@consensys.net>
#154)

Signed-off-by: Christopher Hare <chris.hare@consensys.net>
Signed-off-by: Christopher Hare <chris.hare@consensys.net>
Signed-off-by: Christopher Hare <chris.hare@consensys.net>
Signed-off-by: Christopher Hare <chris.hare@consensys.net>
Signed-off-by: Christopher Hare <chris.hare@consensys.net>

Signed-off-by: Christopher Hare <chris.hare@consensys.net>
Support the "--identity" flag.  This adds a fifth field to the normally 
four part clientId, with the identity in the second position.

For example, if the CLI flag "--identity PegaSysEng" were passed in the
clientID reported by ethernodes would read

`besu/PegaSysEng/v1.3.2/linux-x86_64/oracle_openjdk-java-11`

Whereas without the flag it would just read

`besu/v1.3.2/linux-x86_64/oracle_openjdk-java-11`

Signed-off-by: Danno Ferrin <danno.ferrin@gmail.com>
Signed-off-by: Christopher Hare <chris.hare@consensys.net>
Signed-off-by: Ratan Rai Sur <ratan.r.sur@gmail.com>
Signed-off-by: Christopher Hare <chris.hare@consensys.net>
…or the release tomorrow and dont have jenkins & circle attempting to publish (#157)

Signed-off-by: Joshua Fernandes <joshua.fernandes@consensys.net>
Signed-off-by: Christopher Hare <chris.hare@consensys.net>
@CjHare
Copy link
Contributor Author

CjHare commented Nov 5, 2019

Broken history

@CjHare CjHare closed this Nov 5, 2019
@CjHare CjHare deleted the refactor-jsonrpc-factories branch November 6, 2019 07:21
siladu pushed a commit to siladu/besu that referenced this pull request Oct 28, 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.

6 participants