Extend OperationTracer with new methods #5662

daniellehrner · 2023-06-30T05:49:49Z

PR description

This PR extends the OperationTracer with 2 new methods:

traceStartTransaction, which complements the existing traceEndTransaction
isExtendedTracing, which returns false by default, but can be overrriden by other tracer implementations

Extended tracing is needed in cases where the current tracing implementations does not provide sufficient insight into a transaction. This is for example required for zkEVM implementations as those often require a more detailed insight into a transaction for the proof generation. This PR introduces extended tracing in case of a successful contract creation. More extending tracing will be added in future PRs.

Further the plugin API for tracing is extended as well. It adds 2 new methods:

traceStartBlock
traceEndBlock

Both take a block header and a block body as a parameter. It was required to use those 2 instead of a Block object, because the latter is causing circular dependency issues that would require a very big refactoring to resolve them.

Fixed Issue(s)

github-actions · 2023-06-30T05:49:59Z

I thought about documentation and added the doc-change-required label to this PR if updates are required.
I thought about the changelog and included a changelog update if required.
If my PR includes database changes (e.g. KeyValueSegmentIdentifier) I have thought about compatibility and performed forwards and backwards compatibility tests

evm/src/main/java/org/hyperledger/besu/evm/tracing/OperationTracer.java

...n-api/src/main/java/org/hyperledger/besu/plugin/services/tracer/ExtendedOperationTracer.java

shemnon · 2023-06-30T14:25:33Z

evm/src/main/java/org/hyperledger/besu/evm/processor/ContractCreationProcessor.java

@@ -175,6 +175,9 @@ public void codeSuccess(final MessageFrame frame, final OperationTracer operatio
            contractCode.size(),
            frame.getRemainingGas());
        frame.setState(MessageFrame.State.COMPLETED_SUCCESS);
+        if (operationTracer.isExtendedTracing()) {


Why only in extended? Why not log this account creation all the time?

It was not logged before, only in the failure cases. So it would change the existing behavior for anybody using the current tracer implementation. I think it makes sense to always log it though. So we might just change it to always log it.

Let me know if there are any testing diffs from this. The only place I see it impacting current test code is the trace rpcs.

All the current tests are passing, so there is no problem in general. Do you think it makes sense to log it all the time? I am not sure why we did not do in the first place. But that might just have been an oversight.

...n-api/src/main/java/org/hyperledger/besu/plugin/services/tracer/ExtendedOperationTracer.java

...api/src/main/java/org/hyperledger/besu/plugin/services/tracer/BlockAwareOperationTracer.java

+   * @param blockHeader the header of the block which is traced
+   * @param blockBody the body of the block which is traced
+   */
+  default void traceEndBlock(final BlockHeader blockHeader, final BlockBody blockBody) {}


...api/src/main/java/org/hyperledger/besu/plugin/services/tracer/BlockAwareOperationTracer.java

+   * @param blockHeader the header of the block which is traced
+   * @param blockBody the body of the block which is traced
+   */
+  default void traceEndBlock(final BlockHeader blockHeader, final BlockBody blockBody) {}


...api/src/main/java/org/hyperledger/besu/plugin/services/tracer/BlockAwareOperationTracer.java

+   * @param blockHeader the header of the block which is traced
+   * @param blockBody the body of the block which is traced
+   */
+  default void traceStartBlock(final BlockHeader blockHeader, final BlockBody blockBody) {}


...api/src/main/java/org/hyperledger/besu/plugin/services/tracer/BlockAwareOperationTracer.java

+   * @param blockHeader the header of the block which is traced
+   * @param blockBody the body of the block which is traced
+   */
+  default void traceStartBlock(final BlockHeader blockHeader, final BlockBody blockBody) {}


evm/src/main/java/org/hyperledger/besu/evm/tracing/OperationTracer.java

Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

…acer.java Co-authored-by: Sally MacFarlane <macfarla.github@gmail.com> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

* only update peer with the first and last block Signed-off-by: Stefan <stefan.pingel@consensys.net> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

) * put the download links for 23.4.4 into the CHANGELOG.md * clean up CHANGELOG Signed-off-by: Stefan <stefan.pingel@consensys.net> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

…ledger#5661) We need to replace previous release's latest tag. Might be broken due to a recent change in the docker command implementation Signed-off-by: Simon Dudley <simon.dudley@consensys.net> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

Unrelated: clarify epoch length in javadoc Signed-off-by: Simon Dudley <simon.dudley@consensys.net> Co-authored-by: Gabriel Fukushima <gabrielfukushima@gmail.com> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

) Add hooks for a successful contract create, a failed contract create, and an invalid contact create. Users of the library will be able to customize create responses without having to replace core logic. Signed-off-by: Danno Ferrin <danno.ferrin@swirldslabs.com> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

- support legacy V values (larger V value) and type 1+ (v is recId only) - new fields - shared transaction extraction - rejection detection Signed-off-by: Danno Ferrin <danno.ferrin@swirldslabs.com> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

Upgrade bouncy castle to v1.75. This involved a change in maven coordinates for other modules. Signed-off-by: Danno Ferrin <danno.ferrin@swirldslabs.com> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

Signed-off-by: Fabio Di Fabio <fabio.difabio@consensys.net> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

…er#5669) * Modify the min-gas-price option validation * Check for whether ethash is in use, either from genesis or network config, and use that for miner checks * Add genesis configuration isPoa() convenience function --------- Signed-off-by: Matthew Whitehead <matthew1001@gmail.com> Signed-off-by: Matt Whitehead <matthew1001@hotmail.com> Co-authored-by: Simon Dudley <simon.dudley@consensys.net> Signed-off-by: Simon Dudley <simon.dudley@consensys.net> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

* Revert "Revert "Update Tuweni to 2.4.1 (hyperledger#5513)" (hyperledger#5585)" This reverts commit 6111e1b. Signed-off-by: Antoine Toulme <antoine@lunar-ocean.com> * update Tuweni to 2.4.2 Signed-off-by: Antoine Toulme <antoine@lunar-ocean.com> --------- Signed-off-by: Antoine Toulme <antoine@lunar-ocean.com> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

Signed-off-by: Fabio Di Fabio <fabio.difabio@consensys.net> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

Signed-off-by: Karim TAAM <karim.t2am@gmail.com> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

besu/src/main/java/org/hyperledger/besu/services/TraceServiceImpl.java

...erencetests/src/main/java/org/hyperledger/besu/ethereum/referencetests/ReferenceTestEnv.java

@@ -105,8 +108,14 @@
      @JsonProperty("parentGasUsed") final String parentGasUsed,
      @JsonProperty("parentGasLimit") final String parentGasLimit,
      @JsonProperty("parentTimestamp") final String parentTimestamp,
+      @JsonProperty("ommers") final List<String> _ommers,


...erencetests/src/main/java/org/hyperledger/besu/ethereum/referencetests/ReferenceTestEnv.java

@@ -105,8 +108,14 @@
      @JsonProperty("parentGasUsed") final String parentGasUsed,
      @JsonProperty("parentGasLimit") final String parentGasLimit,
      @JsonProperty("parentTimestamp") final String parentTimestamp,
+      @JsonProperty("ommers") final List<String> _ommers,
+      @JsonProperty("parentUncleHash") final String _parentUncleHash,


...erencetests/src/main/java/org/hyperledger/besu/ethereum/referencetests/ReferenceTestEnv.java

-      @JsonProperty("blockHashes") final Map<String, String> blockHashes) {
+      @JsonProperty("blockHashes") final Map<String, String> blockHashes,
+      @JsonProperty("currentExcessDataGas") final String currentExcessDataGas,
+      @JsonProperty("currentDataGasUsed") final String currentDataGasUsed,


Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

…acer.traceStartTransaction method Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

evm/src/main/java/org/hyperledger/besu/evm/tracing/OperationTracer.java

+   *
+   * @param transaction the transaction which will be processed
+   */
+  default void traceStartTransaction(final Transaction transaction) {}


macfarla

LGTM

shemnon

I'm hesitant to move Transaction into datatypes, but I cannot think of a better alternative that preserves the ability to be concise on method parameters. I am also seeing some advantage of bring the notion of a transaction in a stronger way to the EVM, so there are pluses to be sure.

So with hesitation, LGTM.

* Extend OperationTracer with new methods, create Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> * Rename ExtendedOperationTracer to BlockAwareOperationTracer Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> * fixed plugin api hash Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> * added missing javadoc comments Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> * Update evm/src/main/java/org/hyperledger/besu/evm/tracing/OperationTracer.java Co-authored-by: Sally MacFarlane <macfarla.github@gmail.com> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> * Only update peer with the first and last block (hyperledger#5659) * only update peer with the first and last block Signed-off-by: Stefan <stefan.pingel@consensys.net> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> * Put download links into CHANGELOG and do some clean up (hyperledger#5660) * put the download links for 23.4.4 into the CHANGELOG.md * clean up CHANGELOG Signed-off-by: Stefan <stefan.pingel@consensys.net> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> * Add --amend to docker manifest create to fix docker latest tag (hyperledger#5661) We need to replace previous release's latest tag. Might be broken due to a recent change in the docker command implementation Signed-off-by: Simon Dudley <simon.dudley@consensys.net> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> * Add Xlayered-tx-pool to the config log printout (hyperledger#5665) Unrelated: clarify epoch length in javadoc Signed-off-by: Simon Dudley <simon.dudley@consensys.net> Co-authored-by: Gabriel Fukushima <gabrielfukushima@gmail.com> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> * Add hooks to AbstractCreateOperation for library users (hyperledger#5656) Add hooks for a successful contract create, a failed contract create, and an invalid contact create. Users of the library will be able to customize create responses without having to replace core logic. Signed-off-by: Danno Ferrin <danno.ferrin@swirldslabs.com> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> * Test updates for cancun execution-spec-tests (hyperledger#5670) - support legacy V values (larger V value) and type 1+ (v is recId only) - new fields - shared transaction extraction - rejection detection Signed-off-by: Danno Ferrin <danno.ferrin@swirldslabs.com> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> * Upgrade BouncyCastle libraries (hyperledger#5675) Upgrade bouncy castle to v1.75. This involved a change in maven coordinates for other modules. Signed-off-by: Danno Ferrin <danno.ferrin@swirldslabs.com> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> * Introduce transaction validator interface (phase 1) (hyperledger#5673) Signed-off-by: Fabio Di Fabio <fabio.difabio@consensys.net> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> * Only validate `--miner-enabled` option for ethash networks (hyperledger#5669) * Modify the min-gas-price option validation * Check for whether ethash is in use, either from genesis or network config, and use that for miner checks * Add genesis configuration isPoa() convenience function --------- Signed-off-by: Matthew Whitehead <matthew1001@gmail.com> Signed-off-by: Matt Whitehead <matthew1001@hotmail.com> Co-authored-by: Simon Dudley <simon.dudley@consensys.net> Signed-off-by: Simon Dudley <simon.dudley@consensys.net> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> * Update tuweni2.4.2 (hyperledger#5684) * Revert "Revert "Update Tuweni to 2.4.1 (hyperledger#5513)" (hyperledger#5585)" This reverts commit 6111e1b. Signed-off-by: Antoine Toulme <antoine@lunar-ocean.com> * update Tuweni to 2.4.2 Signed-off-by: Antoine Toulme <antoine@lunar-ocean.com> --------- Signed-off-by: Antoine Toulme <antoine@lunar-ocean.com> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> * Do not leak references to PendingTransactions (hyperledger#5693) Signed-off-by: Fabio Di Fabio <fabio.difabio@consensys.net> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> * remove v0 version of the database (hyperledger#5698) Signed-off-by: Karim TAAM <karim.t2am@gmail.com> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> * updated plugin api hash Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> * spotless Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> * updating plugin api hash Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> * moved transcation interface to datatypes to use it in the OperationTracer.traceStartTransaction method Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> * fix import Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> --------- Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> Signed-off-by: Stefan <stefan.pingel@consensys.net> Signed-off-by: Simon Dudley <simon.dudley@consensys.net> Signed-off-by: Danno Ferrin <danno.ferrin@swirldslabs.com> Signed-off-by: Fabio Di Fabio <fabio.difabio@consensys.net> Signed-off-by: Matthew Whitehead <matthew1001@gmail.com> Signed-off-by: Matt Whitehead <matthew1001@hotmail.com> Signed-off-by: Antoine Toulme <antoine@lunar-ocean.com> Signed-off-by: Karim TAAM <karim.t2am@gmail.com> Co-authored-by: Sally MacFarlane <macfarla.github@gmail.com> Co-authored-by: Stefan Pingel <16143240+pinges@users.noreply.github.com> Co-authored-by: Simon Dudley <simon.dudley@consensys.net> Co-authored-by: Gabriel Fukushima <gabrielfukushima@gmail.com> Co-authored-by: Danno Ferrin <danno.ferrin@gmail.com> Co-authored-by: Fabio Di Fabio <fabio.difabio@consensys.net> Co-authored-by: Matt Whitehead <matthew1001@gmail.com> Co-authored-by: Antoine Toulme <antoine@lunar-ocean.com> Co-authored-by: matkt <karim.t2am@gmail.com>

This reverts commit bb8514d.

* Extend OperationTracer with new methods, create Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> * Rename ExtendedOperationTracer to BlockAwareOperationTracer Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> * fixed plugin api hash Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> * added missing javadoc comments Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> * Update evm/src/main/java/org/hyperledger/besu/evm/tracing/OperationTracer.java Co-authored-by: Sally MacFarlane <macfarla.github@gmail.com> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> * Only update peer with the first and last block (hyperledger#5659) * only update peer with the first and last block Signed-off-by: Stefan <stefan.pingel@consensys.net> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> * Put download links into CHANGELOG and do some clean up (hyperledger#5660) * put the download links for 23.4.4 into the CHANGELOG.md * clean up CHANGELOG Signed-off-by: Stefan <stefan.pingel@consensys.net> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> * Add --amend to docker manifest create to fix docker latest tag (hyperledger#5661) We need to replace previous release's latest tag. Might be broken due to a recent change in the docker command implementation Signed-off-by: Simon Dudley <simon.dudley@consensys.net> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> * Add Xlayered-tx-pool to the config log printout (hyperledger#5665) Unrelated: clarify epoch length in javadoc Signed-off-by: Simon Dudley <simon.dudley@consensys.net> Co-authored-by: Gabriel Fukushima <gabrielfukushima@gmail.com> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> * Add hooks to AbstractCreateOperation for library users (hyperledger#5656) Add hooks for a successful contract create, a failed contract create, and an invalid contact create. Users of the library will be able to customize create responses without having to replace core logic. Signed-off-by: Danno Ferrin <danno.ferrin@swirldslabs.com> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> * Test updates for cancun execution-spec-tests (hyperledger#5670) - support legacy V values (larger V value) and type 1+ (v is recId only) - new fields - shared transaction extraction - rejection detection Signed-off-by: Danno Ferrin <danno.ferrin@swirldslabs.com> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> * Upgrade BouncyCastle libraries (hyperledger#5675) Upgrade bouncy castle to v1.75. This involved a change in maven coordinates for other modules. Signed-off-by: Danno Ferrin <danno.ferrin@swirldslabs.com> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> * Introduce transaction validator interface (phase 1) (hyperledger#5673) Signed-off-by: Fabio Di Fabio <fabio.difabio@consensys.net> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> * Only validate `--miner-enabled` option for ethash networks (hyperledger#5669) * Modify the min-gas-price option validation * Check for whether ethash is in use, either from genesis or network config, and use that for miner checks * Add genesis configuration isPoa() convenience function --------- Signed-off-by: Matthew Whitehead <matthew1001@gmail.com> Signed-off-by: Matt Whitehead <matthew1001@hotmail.com> Co-authored-by: Simon Dudley <simon.dudley@consensys.net> Signed-off-by: Simon Dudley <simon.dudley@consensys.net> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> * Update tuweni2.4.2 (hyperledger#5684) * Revert "Revert "Update Tuweni to 2.4.1 (hyperledger#5513)" (hyperledger#5585)" This reverts commit 6111e1b. Signed-off-by: Antoine Toulme <antoine@lunar-ocean.com> * update Tuweni to 2.4.2 Signed-off-by: Antoine Toulme <antoine@lunar-ocean.com> --------- Signed-off-by: Antoine Toulme <antoine@lunar-ocean.com> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> * Do not leak references to PendingTransactions (hyperledger#5693) Signed-off-by: Fabio Di Fabio <fabio.difabio@consensys.net> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> * remove v0 version of the database (hyperledger#5698) Signed-off-by: Karim TAAM <karim.t2am@gmail.com> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> * updated plugin api hash Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> * spotless Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> * updating plugin api hash Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> * moved transcation interface to datatypes to use it in the OperationTracer.traceStartTransaction method Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> * fix import Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> --------- Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net> Signed-off-by: Stefan <stefan.pingel@consensys.net> Signed-off-by: Simon Dudley <simon.dudley@consensys.net> Signed-off-by: Danno Ferrin <danno.ferrin@swirldslabs.com> Signed-off-by: Fabio Di Fabio <fabio.difabio@consensys.net> Signed-off-by: Matthew Whitehead <matthew1001@gmail.com> Signed-off-by: Matt Whitehead <matthew1001@hotmail.com> Signed-off-by: Antoine Toulme <antoine@lunar-ocean.com> Signed-off-by: Karim TAAM <karim.t2am@gmail.com> Co-authored-by: Sally MacFarlane <macfarla.github@gmail.com> Co-authored-by: Stefan Pingel <16143240+pinges@users.noreply.github.com> Co-authored-by: Simon Dudley <simon.dudley@consensys.net> Co-authored-by: Gabriel Fukushima <gabrielfukushima@gmail.com> Co-authored-by: Danno Ferrin <danno.ferrin@gmail.com> Co-authored-by: Fabio Di Fabio <fabio.difabio@consensys.net> Co-authored-by: Matt Whitehead <matthew1001@gmail.com> Co-authored-by: Antoine Toulme <antoine@lunar-ocean.com> Co-authored-by: matkt <karim.t2am@gmail.com>

github-advanced-security bot found potential problems Jun 30, 2023

View reviewed changes

shemnon reviewed Jun 30, 2023

View reviewed changes

github-advanced-security bot found potential problems Jul 5, 2023

View reviewed changes

macfarla reviewed Jul 10, 2023

View reviewed changes

evm/src/main/java/org/hyperledger/besu/evm/tracing/OperationTracer.java Outdated Show resolved Hide resolved

non-fungible-nelson added the TeamChupa GH issues worked on by Chupacabara Team label Jul 11, 2023

daniellehrner and others added 17 commits July 13, 2023 17:39

Extend OperationTracer with new methods, create

b23b687

Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

Rename ExtendedOperationTracer to BlockAwareOperationTracer

6dc1036

Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

fixed plugin api hash

686543a

Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

added missing javadoc comments

462fc7b

Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

Update evm/src/main/java/org/hyperledger/besu/evm/tracing/OperationTr…

a4f68e3

…acer.java Co-authored-by: Sally MacFarlane <macfarla.github@gmail.com> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

Only update peer with the first and last block (hyperledger#5659)

ca08143

* only update peer with the first and last block Signed-off-by: Stefan <stefan.pingel@consensys.net> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

Put download links into CHANGELOG and do some clean up (hyperledger#5660

aae47b5

) * put the download links for 23.4.4 into the CHANGELOG.md * clean up CHANGELOG Signed-off-by: Stefan <stefan.pingel@consensys.net> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

Upgrade BouncyCastle libraries (hyperledger#5675)

148e243

Upgrade bouncy castle to v1.75. This involved a change in maven coordinates for other modules. Signed-off-by: Danno Ferrin <danno.ferrin@swirldslabs.com> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

Introduce transaction validator interface (phase 1) (hyperledger#5673)

35655c7

Signed-off-by: Fabio Di Fabio <fabio.difabio@consensys.net> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

Do not leak references to PendingTransactions (hyperledger#5693)

ab4e321

Signed-off-by: Fabio Di Fabio <fabio.difabio@consensys.net> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

remove v0 version of the database (hyperledger#5698)

ccab3fb

Signed-off-by: Karim TAAM <karim.t2am@gmail.com> Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

daniellehrner force-pushed the extend_operation_tracer branch from 0a1c407 to ccab3fb Compare July 13, 2023 15:39

updated plugin api hash

dfbbbdf

Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

delehef reviewed Jul 13, 2023

View reviewed changes

besu/src/main/java/org/hyperledger/besu/services/TraceServiceImpl.java Outdated Show resolved Hide resolved

github-advanced-security bot found potential problems Jul 13, 2023

View reviewed changes

daniellehrner added 3 commits July 14, 2023 11:06

spotless

ef72b08

Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

updating plugin api hash

a9a9f91

Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

moved transcation interface to datatypes to use it in the OperationTr…

722d56f

…acer.traceStartTransaction method Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

daniellehrner added 2 commits July 14, 2023 13:06

Merge branch 'main' into extend_operation_tracer

1930494

Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

fix import

65ad951

Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>

github-advanced-security bot found potential problems Jul 14, 2023

View reviewed changes

macfarla approved these changes Jul 17, 2023

View reviewed changes

shemnon approved these changes Jul 17, 2023

View reviewed changes

Merge branch 'main' into extend_operation_tracer

b6f330d

daniellehrner enabled auto-merge (squash) July 20, 2023 10:48

daniellehrner merged commit 1a08770 into hyperledger:main Jul 20, 2023
19 checks passed

daniellehrner deleted the extend_operation_tracer branch July 20, 2023 11:56

davidkngo added a commit to liquichain/besu that referenced this pull request Jul 21, 2023

Revert "Extend OperationTracer with new methods (hyperledger#5662)"

d1eebee

This reverts commit bb8514d.

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Extend OperationTracer with new methods #5662

Extend OperationTracer with new methods #5662

daniellehrner commented Jun 30, 2023

github-actions bot commented Jun 30, 2023

shemnon Jun 30, 2023

daniellehrner Jul 5, 2023

shemnon Jul 5, 2023

daniellehrner Jul 14, 2023

macfarla left a comment

shemnon left a comment

Extend OperationTracer with new methods #5662

Extend OperationTracer with new methods #5662

Conversation

daniellehrner commented Jun 30, 2023

PR description

Fixed Issue(s)

github-actions bot commented Jun 30, 2023

shemnon Jun 30, 2023

Choose a reason for hiding this comment

daniellehrner Jul 5, 2023

Choose a reason for hiding this comment

shemnon Jul 5, 2023

Choose a reason for hiding this comment

daniellehrner Jul 14, 2023

Choose a reason for hiding this comment

macfarla left a comment

Choose a reason for hiding this comment

shemnon left a comment

Choose a reason for hiding this comment