Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Transactions hashes missing in trace_replayBlockTransactions method result #8725

Closed
adirmola opened this issue May 29, 2018 · 14 comments · Fixed by #8883
Closed

Transactions hashes missing in trace_replayBlockTransactions method result #8725

adirmola opened this issue May 29, 2018 · 14 comments · Fixed by #8883
Labels
F8-enhancement 🎊 An additional feature request. M6-rpcapi 📣 RPC API. P7-nicetohave 🐕 Issue is worth doing eventually. Q3-medium A fair chunk of work, not necessarily very hard but not trivial either
Milestone

Comments

@adirmola
Copy link

adirmola commented May 29, 2018

Before filing a new issue, please provide the following information.

I'm running:

  • Which Parity version?: 1.12.0
  • Which operating system?: Windows
  • How installed?: via installer
  • Are you fully synchronized?: yes
  • Which network are you connected to?: ropsten
  • Did you try to restart the node?: yes

Hi,

When running the method "trace_replayBlockTransactions" with "stateDiff" parameter I'm getting back a list of stateDiff of all transaction in a specific block.
The problem is that the result doesn't include the transactions hashes so we cant link the specific stateDiff to the related transaction.

For example, if running (on ropsten) the following:
curl --data '{"method":"trace_replayBlockTransactions","params":["0x32d8e6",["trace"]],"id":1,"jsonrpc":"2.0"}' -H "Content-Type: application/json" -X POST localhost:8545

I'm getting:

{
    'jsonrpc':  '2.0',
     'result':  [
        {
            'output':  '0x000000000000000000000000202934ddeda0d38f30a70e9cf888c13afc2792bc',
             'stateDiff':  {
                '0x202934ddeda0d38f30a70e9cf888c13afc2792bc':  {
                    'balance':  { '+':  '0x0'},
                     'code':  { '+':  '0x6080..."},
                     'nonce':  { '+':  '0x1'},
                     'storage':  {
                        '0x0000000000000000000000000000000000000000000000000000000000000000':  {
                            '+':  '0x000000000000000000000000e7a043b5db22472df387d3cca173b6227b37f1f4' }
                    }
                },
                 '0xb2d9a807815d6e9afff805b835cd06cb990db423':  {
                    'balance':  '=',
                     'code':  '=',
                     'nonce':  { '*':  {'from':  '0x7', 'to':  '0x8' }},
                     'storage':  { }
                },
                 '0xe24246e6dcbb07bc15a1f9c3833fc1877df4c80e':  {
                    'balance':  { '*':  { 'from':  '0x7c5e833076332a74ad2','to':  '0x7c5e835123266a208d2' } },
                     'code':  '=',
                     'nonce':  '=',
                     'storage':  {}
                },
                 '0xe7a043b5db22472df387d3cca173b6227b37f1f4':  {
                    'balance':  { '*':  { 'from':  '0xd53e81b3de42cd4','to':  '0xd51dd4c09e96ed4' }},
                     'code':  '=',
                     'nonce':  { '*':  {'from':  '0x2a', 'to':  '0x2b' } },
                     'storage':  { }
                }
            },
             'trace':  [],
             'vmTrace':  None
        },
....],
     'id':  1
}

which you can see doesn't include the transaction hash.

Meanwhile (after checking with parity team on gitter) we can assume that the result of "trace_replayBlockTransactions" is in order with transactions index on the block.
Obvious, it isn't the straightforward way to handle this issue.

So adding the transaction hash to the result will be great.

@Tbaut Tbaut added F8-enhancement 🎊 An additional feature request. M6-rpcapi 📣 RPC API. Q3-medium A fair chunk of work, not necessarily very hard but not trivial either labels May 29, 2018
@Tbaut Tbaut added this to the 1.12 milestone May 29, 2018
@Tbaut Tbaut added the P7-nicetohave 🐕 Issue is worth doing eventually. label May 29, 2018
@gitcoinbot
Copy link

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


This issue now has a funding of 0.15 ETH (85.92 USD @ $572.77/ETH) attached to it.

@gitcoinbot
Copy link

gitcoinbot commented Jun 4, 2018

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


Work has been started.

These users each claimed they can complete the work by 8 months, 2 weeks ago.
Please review their action plans below:

1) shamardy has started work.

working on this

Learn more on the Gitcoin Issue Details page.

@vs77bb
Copy link

vs77bb commented Jun 6, 2018

@shamardy Hello - any update on your side? Would appreciate a WIP PR for review whenever is possible 🙂

@shamardy
Copy link
Contributor

shamardy commented Jun 7, 2018

@vs77bb sorry for the delay it took me some time to run the parity node.

I ran parity using
parity --chain=ropsten --pruning=archive --jsonrpc-apis traces --base-path ./foobar

then I ran the following command after syncing with the block
curl -X POST -H "Content-Type: application/json" -o "output.txt" --data "{\"jsonrpc\":\"2.0\",\"id\":1,\"method\":\"trace_replayBlockTransactions\",\"params\":[\"0x8265\",[\"trace\"]]}" http://127.0.0.1:8545

and I got a normal output that shows transactions
{"jsonrpc":"2.0","result":[{"output":"0x","stateDiff":null,"trace":[{"action":{"callType":"call","from":"0x13c0127b66b336a644cc36c2e44eadc5fcd8b79d","gas":"0x37b38","input":"0x51a34eb80000000000000000000000000000000000000000000000007ce647efa8b98000","to":"0x06eadbd0b793d7614bf42d8d09980bf345ec748b","value":"0x0"},"result":{"gasUsed":"0x12bb3","output":"0x"},"subtraces":2,"traceAddress":[],"type":"call"},{"action":{"callType":"call","from":"0x06eadbd0b793d7614bf42d8d09980bf345ec748b","gas":"0x31217","input":"0xe16c7d98636f6e7472616374617069000000000000000000000000000000000000000000","to":"0x2cccf5e0538493c235d1c5ef6580f77d99e91396","value":"0x0"},"result":{"gasUsed":"0x334","output":"0x000000000000000000000000b4fe7aa695b326c9d219158d2ca50db77b39f99f"},"subtraces":0,"traceAddress":[0],"type":"call"},{"action":{"callType":"call","from":"0x06eadbd0b793d7614bf42d8d09980bf345ec748b","gas":"0x30b4a","input":"0x51a34eb80000000000000000000000000000000000000000000000007ce647efa8b98000","to":"0xb4fe7aa695b326c9d219158d2ca50db77b39f99f","value":"0x0"},"result":{"gasUsed":"0x24d","output":"0x0000000000000000000000000000000000000000000000000000000000000001"},"subtraces":0,"traceAddress":[1,3,11,0],"type":"call"}],"vmTrace":null}],"id":1}

I didn't query block "0x32d8e6" the same as @adirmola because it would take me some time to sync to it. Maybe this block had no transactions?

@shamardy
Copy link
Contributor

shamardy commented Jun 7, 2018

sorry I forgot to write the results with stateDiff, for this I ran this command:
curl -X POST -H "Content-Type: application/json" -o "output.txt" --data "{\"jsonrpc\":\"2.0\",\"id\":1,\"method\":\"trace_replayBlockTransactions\",\"params\":[\"0x8265\",[\"trace\",\"stateDiff\"]]}" http://127.0.0.1:8545

results:
{"jsonrpc":"2.0","result":[{"output":"0x","stateDiff":{"0x06eadbd0b793d7614bf42d8d09980bf345ec748b":{"balance":"=","code":"=","nonce":"=","storage":{"0x0000000000000000000000000000000000000000000000000000000000000002":{"*":{"from":"0x0000000000000000000000000000000000000000000000000000000000008255","to":"0x0000000000000000000000000000000000000000000000000000000000008265"}},"0x000000000000000000000000000000000000000000000000000000000000000b":{"*":{"from":"0x0000000000000000000000000000000000000000000000007ce66c50e2840000","to":"0x0000000000000000000000000000000000000000000000007ce647efa8b98000"}}}},"0x13c0127b66b336a644cc36c2e44eadc5fcd8b79d":{"balance":{"*":{"from":"0x2da34a0533deaff85","to":"0x2da2d9f54411d6785"}},"code":"=","nonce":{"*":{"from":"0x9fd","to":"0x9fe"}},"storage":{}},"0xcf00ffd997ad14939736f026006498e3f099baaf":{"balance":"=","code":"=","nonce":"=","storage":{"0x070c259bc5bb5442a0e39365770918681724c9f5cca6888e9ff72def394851f7":{"*":{"from":"0x0000000000000000000000000000000000000000000000185de5cf8fa04dc000","to":"0x0000000000000000000000000000000000000000000000185de463c35e64c000"}}}},"0xfcb9fb8d0e02d1dee561d2534c821af2ad8a5d13":{"balance":{"*":{"from":"0x1035baf0b0016f7db1b","to":"0x1035bb60bff13c5731b"}},"code":"=","nonce":"=","storage":{}}},"trace":[],"vmTrace":null}],"id":1}

Again It has the transactions

@adirmola
Copy link
Author

adirmola commented Jun 7, 2018

Hey @shamardy,
I think u missing the point.
I didn't say there isn't transactions but the transactions don't retrieved with their hashes.

@gitcoinbot
Copy link

@shamardy Hello from Gitcoin Core - are you still working on this issue? Please submit a WIP PR or comment back within the next 3 days or you will be removed from this ticket and it will be returned to an ‘Open’ status. Please let us know if you have questions!

  • warning (3 days)
  • escalation to mods (6 days)

Funders only: Snooze warnings for 1 day | 3 days | 5 days | 10 days | 100 days

shamardy added a commit to shamardy/parity that referenced this issue Jun 13, 2018
Transactions hashes missing in trace_replayBlockTransactions method result openethereum#8725
@shamardy
Copy link
Contributor

submitted a pull request:
#8883

shamardy added a commit to shamardy/parity that referenced this issue Jun 14, 2018
Transactions hashes missing in trace_replayBlockTransactions method result openethereum#8725
@gitcoinbot
Copy link

@shamardy Hello from Gitcoin Core - are you still working on this issue? Please submit a WIP PR or comment back within the next 3 days or you will be removed from this ticket and it will be returned to an ‘Open’ status. Please let us know if you have questions!

  • warning (3 days)
  • escalation to mods (6 days)

Funders only: Snooze warnings for 1 day | 3 days | 5 days | 10 days | 100 days

@shamardy
Copy link
Contributor

Yes, I am still working on this issue, waiting for response from @debris in #8883

@vs77bb
Copy link

vs77bb commented Jun 21, 2018

Thanks for the update @shamardy - snoozing Gitcoin bot for a bit here.

shamardy added a commit to shamardy/parity that referenced this issue Jul 1, 2018
commit 1f85076
Author: shamardy <shamardy@yahoo.com>
Date:   Sat Jun 30 03:40:12 2018 +0200

    update

commit 63363cc
Author: shamardy <shamardy@yahoo.com>
Date:   Sat Jun 30 03:36:25 2018 +0200

    Update

commit e05cadd
Author: shamardy <shamardy@yahoo.com>
Date:   Fri Jun 29 10:04:34 2018 +0200

    Test

commit b0be065
Author: shamardy <shamardy@yahoo.com>
Date:   Fri Jun 29 09:19:57 2018 +0200

    test_client edit

commit 949205c
Author: shamardy <shamardy@yahoo.com>
Date:   Fri Jun 29 05:14:53 2018 +0200

    Edit Test

commit 7cd44ee
Author: shamardy <shamardy@yahoo.com>
Date:   Fri Jun 29 04:25:22 2018 +0200

    Updates

commit e90de71
Author: shamardy <shamardy@yahoo.com>
Date:   Wed Jun 27 13:53:15 2018 +0200

    Test

commit 12a7638
Author: shamardy <shamardy@yahoo.com>
Date:   Wed Jun 27 11:43:39 2018 +0200

    Edited Tests

commit 6c21e6d
Merge: 87c4c74 9550cf7
Author: shamardy <shamardy@yahoo.com>
Date:   Wed Jun 27 10:31:21 2018 +0200

    Merge branch 'Issue#8725' of https://github.com/shamardy/parity into Issue#8725

commit 87c4c74
Author: shamardy <shamardy@yahoo.com>
Date:   Wed Jun 27 10:31:13 2018 +0200

    Avoided Changing non-RPC Types

commit 9550cf7
Author: shamardy <39480341+shamardy@users.noreply.github.com>
Date:   Wed Jun 27 10:29:03 2018 +0200

    Update traces.rs

commit 3e0b0ef
Author: shamardy <39480341+shamardy@users.noreply.github.com>
Date:   Wed Jun 27 10:27:44 2018 +0200

    Update trace.rs

commit 5078d67
Author: shamardy <39480341+shamardy@users.noreply.github.com>
Date:   Wed Jun 27 10:26:08 2018 +0200

    Update traces.rs

commit 28f5ba1
Author: shamardy <39480341+shamardy@users.noreply.github.com>
Date:   Wed Jun 27 10:24:04 2018 +0200

    Update parity.rs

commit 3b86b98
Author: shamardy <39480341+shamardy@users.noreply.github.com>
Date:   Wed Jun 27 10:23:33 2018 +0200

    Update eth.rs

commit bcad5a4
Author: shamardy <39480341+shamardy@users.noreply.github.com>
Date:   Wed Jun 27 10:21:43 2018 +0200

    Update call_analytics.rs

commit e9029e0
Author: shamardy <39480341+shamardy@users.noreply.github.com>
Date:   Wed Jun 27 10:20:56 2018 +0200

    Update transaction.rs

commit 0dacc81
Author: shamardy <39480341+shamardy@users.noreply.github.com>
Date:   Wed Jun 27 10:20:01 2018 +0200

    Update executive.rs

commit 3921d4e
Author: shamardy <39480341+shamardy@users.noreply.github.com>
Date:   Wed Jun 27 10:18:39 2018 +0200

    Update executed.rs

commit 8416df6
Author: shamardy <39480341+shamardy@users.noreply.github.com>
Date:   Wed Jun 27 10:17:34 2018 +0200

    Update client.rs

commit cf0b4dd
Merge: 23bfa78 36e05e5
Author: shamardy <shamardy@yahoo.com>
Date:   Sun Jun 24 04:54:04 2018 +0200

    Merge branch 'Issue#8725' of https://github.com/shamardy/parity into Issue#8725

commit 23bfa78
Author: shamardy <shamardy@yahoo.com>
Date:   Sun Jun 24 04:53:50 2018 +0200

    Undo

commit 36e05e5
Merge: 2f6e1ef 0afc748
Author: shamardy <shamardy@yahoo.com>
Date:   Sun Jun 24 04:44:33 2018 +0200

    Merge branch 'Issue#8725' of https://github.com/shamardy/parity into Issue#8725

commit 2f6e1ef
Author: shamardy <shamardy@yahoo.com>
Date:   Sun Jun 24 04:44:22 2018 +0200

    Another rpc test output with ("transactionHash":null)

commit 0afc748
Author: shamardy <shamardy@yahoo.com>
Date:   Thu Jun 14 07:18:45 2018 +0200

    Another rpc test output with ("transactionHash":null)

commit 138fbac
Author: shamardy <shamardy@yahoo.com>
Date:   Thu Jun 14 06:30:41 2018 +0200

    Edited some rpc tests output with ("transactionHash":null)

commit 8c129a6
Author: shamardy <39480341+shamardy@users.noreply.github.com>
Date:   Thu Jun 14 05:48:39 2018 +0200

    Update traces.rs

commit 52c17f6
Author: shamardy <39480341+shamardy@users.noreply.github.com>
Date:   Thu Jun 14 05:47:29 2018 +0200

    Update parity.rs

commit d39303a
Author: shamardy <39480341+shamardy@users.noreply.github.com>
Date:   Thu Jun 14 05:46:25 2018 +0200

    Update eth.rs

commit 49be84b
Author: shamardy <39480341+shamardy@users.noreply.github.com>
Date:   Thu Jun 14 05:40:56 2018 +0200

    Update trace.rs

commit 4fc8013
Merge: d34ba23 5442104
Author: shamardy <shamardy@yahoo.com>
Date:   Thu Jun 14 05:12:21 2018 +0200

    Merge branch 'Issue#8725' of https://github.com/shamardy/parity into Issue#8725

commit 5442104
Author: shamardy <39480341+shamardy@users.noreply.github.com>
Date:   Thu Jun 14 05:04:51 2018 +0200

    Update trace.rs

commit 2b2524a
Author: shamardy <39480341+shamardy@users.noreply.github.com>
Date:   Thu Jun 14 05:01:47 2018 +0200

    Update state_diff.rs

commit 2bf9982
Author: shamardy <39480341+shamardy@users.noreply.github.com>
Date:   Thu Jun 14 05:00:50 2018 +0200

    Update transaction.rs

commit da696ea
Author: shamardy <39480341+shamardy@users.noreply.github.com>
Date:   Thu Jun 14 05:00:13 2018 +0200

    Update mod.rs

commit cfc194c
Author: shamardy <39480341+shamardy@users.noreply.github.com>
Date:   Thu Jun 14 04:59:18 2018 +0200

    Update pod_state.rs

commit 3b31568
Author: shamardy <39480341+shamardy@users.noreply.github.com>
Date:   Thu Jun 14 04:57:37 2018 +0200

    Update client.rs

commit d34ba23
Author: shamardy <shamardy@yahoo.com>
Date:   Thu Jun 14 04:53:50 2018 +0200

    RPC Only

commit 8b5c4f1
Merge: 7444916 291b4a0
Author: shamardy <shamardy@yahoo.com>
Date:   Thu Jun 14 02:55:14 2018 +0200

    Merge branch 'Issue#8725' of https://github.com/shamardy/parity into Issue#8725

commit 7444916
Author: shamardy <shamardy@yahoo.com>
Date:   Thu Jun 14 02:27:13 2018 +0200

    Issue#8725

    Transactions hashes missing in trace_replayBlockTransactions method result openethereum#8725

commit 291b4a0
Author: shamardy <shamardy@yahoo.com>
Date:   Thu Jun 14 01:18:18 2018 +0200

    Edited to make changes to RPC types only

    To make a transactions hash trace with "trace_replayBlockTransactions" add "transactionHash" to parameters

commit 9d082be
Author: shamardy <shamardy@yahoo.com>
Date:   Wed Jun 13 09:15:56 2018 +0200

    Issue#8725

    Transactions hashes missing in trace_replayBlockTransactions method result openethereum#8725
ascjones pushed a commit that referenced this issue Jul 10, 2018
…esult #8725 (#8883)

* Squashed commit of the following:

commit 1f85076
Author: shamardy <shamardy@yahoo.com>
Date:   Sat Jun 30 03:40:12 2018 +0200

    update

commit 63363cc
Author: shamardy <shamardy@yahoo.com>
Date:   Sat Jun 30 03:36:25 2018 +0200

    Update

commit e05cadd
Author: shamardy <shamardy@yahoo.com>
Date:   Fri Jun 29 10:04:34 2018 +0200

    Test

commit b0be065
Author: shamardy <shamardy@yahoo.com>
Date:   Fri Jun 29 09:19:57 2018 +0200

    test_client edit

commit 949205c
Author: shamardy <shamardy@yahoo.com>
Date:   Fri Jun 29 05:14:53 2018 +0200

    Edit Test

commit 7cd44ee
Author: shamardy <shamardy@yahoo.com>
Date:   Fri Jun 29 04:25:22 2018 +0200

    Updates

commit e90de71
Author: shamardy <shamardy@yahoo.com>
Date:   Wed Jun 27 13:53:15 2018 +0200

    Test

commit 12a7638
Author: shamardy <shamardy@yahoo.com>
Date:   Wed Jun 27 11:43:39 2018 +0200

    Edited Tests

commit 6c21e6d
Merge: 87c4c74 9550cf7
Author: shamardy <shamardy@yahoo.com>
Date:   Wed Jun 27 10:31:21 2018 +0200

    Merge branch 'Issue#8725' of https://github.com/shamardy/parity into Issue#8725

commit 87c4c74
Author: shamardy <shamardy@yahoo.com>
Date:   Wed Jun 27 10:31:13 2018 +0200

    Avoided Changing non-RPC Types

commit 9550cf7
Author: shamardy <39480341+shamardy@users.noreply.github.com>
Date:   Wed Jun 27 10:29:03 2018 +0200

    Update traces.rs

commit 3e0b0ef
Author: shamardy <39480341+shamardy@users.noreply.github.com>
Date:   Wed Jun 27 10:27:44 2018 +0200

    Update trace.rs

commit 5078d67
Author: shamardy <39480341+shamardy@users.noreply.github.com>
Date:   Wed Jun 27 10:26:08 2018 +0200

    Update traces.rs

commit 28f5ba1
Author: shamardy <39480341+shamardy@users.noreply.github.com>
Date:   Wed Jun 27 10:24:04 2018 +0200

    Update parity.rs

commit 3b86b98
Author: shamardy <39480341+shamardy@users.noreply.github.com>
Date:   Wed Jun 27 10:23:33 2018 +0200

    Update eth.rs

commit bcad5a4
Author: shamardy <39480341+shamardy@users.noreply.github.com>
Date:   Wed Jun 27 10:21:43 2018 +0200

    Update call_analytics.rs

commit e9029e0
Author: shamardy <39480341+shamardy@users.noreply.github.com>
Date:   Wed Jun 27 10:20:56 2018 +0200

    Update transaction.rs

commit 0dacc81
Author: shamardy <39480341+shamardy@users.noreply.github.com>
Date:   Wed Jun 27 10:20:01 2018 +0200

    Update executive.rs

commit 3921d4e
Author: shamardy <39480341+shamardy@users.noreply.github.com>
Date:   Wed Jun 27 10:18:39 2018 +0200

    Update executed.rs

commit 8416df6
Author: shamardy <39480341+shamardy@users.noreply.github.com>
Date:   Wed Jun 27 10:17:34 2018 +0200

    Update client.rs

commit cf0b4dd
Merge: 23bfa78 36e05e5
Author: shamardy <shamardy@yahoo.com>
Date:   Sun Jun 24 04:54:04 2018 +0200

    Merge branch 'Issue#8725' of https://github.com/shamardy/parity into Issue#8725

commit 23bfa78
Author: shamardy <shamardy@yahoo.com>
Date:   Sun Jun 24 04:53:50 2018 +0200

    Undo

commit 36e05e5
Merge: 2f6e1ef 0afc748
Author: shamardy <shamardy@yahoo.com>
Date:   Sun Jun 24 04:44:33 2018 +0200

    Merge branch 'Issue#8725' of https://github.com/shamardy/parity into Issue#8725

commit 2f6e1ef
Author: shamardy <shamardy@yahoo.com>
Date:   Sun Jun 24 04:44:22 2018 +0200

    Another rpc test output with ("transactionHash":null)

commit 0afc748
Author: shamardy <shamardy@yahoo.com>
Date:   Thu Jun 14 07:18:45 2018 +0200

    Another rpc test output with ("transactionHash":null)

commit 138fbac
Author: shamardy <shamardy@yahoo.com>
Date:   Thu Jun 14 06:30:41 2018 +0200

    Edited some rpc tests output with ("transactionHash":null)

commit 8c129a6
Author: shamardy <39480341+shamardy@users.noreply.github.com>
Date:   Thu Jun 14 05:48:39 2018 +0200

    Update traces.rs

commit 52c17f6
Author: shamardy <39480341+shamardy@users.noreply.github.com>
Date:   Thu Jun 14 05:47:29 2018 +0200

    Update parity.rs

commit d39303a
Author: shamardy <39480341+shamardy@users.noreply.github.com>
Date:   Thu Jun 14 05:46:25 2018 +0200

    Update eth.rs

commit 49be84b
Author: shamardy <39480341+shamardy@users.noreply.github.com>
Date:   Thu Jun 14 05:40:56 2018 +0200

    Update trace.rs

commit 4fc8013
Merge: d34ba23 5442104
Author: shamardy <shamardy@yahoo.com>
Date:   Thu Jun 14 05:12:21 2018 +0200

    Merge branch 'Issue#8725' of https://github.com/shamardy/parity into Issue#8725

commit 5442104
Author: shamardy <39480341+shamardy@users.noreply.github.com>
Date:   Thu Jun 14 05:04:51 2018 +0200

    Update trace.rs

commit 2b2524a
Author: shamardy <39480341+shamardy@users.noreply.github.com>
Date:   Thu Jun 14 05:01:47 2018 +0200

    Update state_diff.rs

commit 2bf9982
Author: shamardy <39480341+shamardy@users.noreply.github.com>
Date:   Thu Jun 14 05:00:50 2018 +0200

    Update transaction.rs

commit da696ea
Author: shamardy <39480341+shamardy@users.noreply.github.com>
Date:   Thu Jun 14 05:00:13 2018 +0200

    Update mod.rs

commit cfc194c
Author: shamardy <39480341+shamardy@users.noreply.github.com>
Date:   Thu Jun 14 04:59:18 2018 +0200

    Update pod_state.rs

commit 3b31568
Author: shamardy <39480341+shamardy@users.noreply.github.com>
Date:   Thu Jun 14 04:57:37 2018 +0200

    Update client.rs

commit d34ba23
Author: shamardy <shamardy@yahoo.com>
Date:   Thu Jun 14 04:53:50 2018 +0200

    RPC Only

commit 8b5c4f1
Merge: 7444916 291b4a0
Author: shamardy <shamardy@yahoo.com>
Date:   Thu Jun 14 02:55:14 2018 +0200

    Merge branch 'Issue#8725' of https://github.com/shamardy/parity into Issue#8725

commit 7444916
Author: shamardy <shamardy@yahoo.com>
Date:   Thu Jun 14 02:27:13 2018 +0200

    Issue#8725

    Transactions hashes missing in trace_replayBlockTransactions method result #8725

commit 291b4a0
Author: shamardy <shamardy@yahoo.com>
Date:   Thu Jun 14 01:18:18 2018 +0200

    Edited to make changes to RPC types only

    To make a transactions hash trace with "trace_replayBlockTransactions" add "transactionHash" to parameters

commit 9d082be
Author: shamardy <shamardy@yahoo.com>
Date:   Wed Jun 13 09:15:56 2018 +0200

    Issue#8725

    Transactions hashes missing in trace_replayBlockTransactions method result #8725

* update

* Removed unwrap_or

* Update

* test_client update
@gitcoinbot
Copy link

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


Work for 0.15 ETH (66.04 USD @ $440.25/ETH) has been submitted by:

  1. @shamardy

@vs77bb please take a look at the submitted work:


@shamardy
Copy link
Contributor

@vs77bb can you please approve this. Thanks.

@gitcoinbot
Copy link

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


The funding of 0.15 ETH (74.61 USD @ $497.38/ETH) attached to this issue has been approved & issued to @shamardy.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
F8-enhancement 🎊 An additional feature request. M6-rpcapi 📣 RPC API. P7-nicetohave 🐕 Issue is worth doing eventually. Q3-medium A fair chunk of work, not necessarily very hard but not trivial either
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants