Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

fix bug in trace_api_plugin no params returned GH issue 10435 / EPE 1066 https://github.com/EOSIO/eos/issues/10435 #10449

Merged
merged 4 commits into from
Jun 23, 2021

Conversation

praphael
Copy link

@praphael praphael commented Jun 23, 2021

Change Description

In trace API plugin, if RPC ABIS are specified with --trace-rpc-abi, the 'params' field of the returned transaction JSON did not exist. This was caused by an exception being thrown in chain::abi_serializer, when the return_value field did not exist, it would thrown an 'Unpack exception' even though action params were serialized correctly. Fixes issue #10435

Change Type

Select ONE:

  • Documentation
  • [ x] Stability bug fix
  • Other
  • Other - special case

Testing Changes

Select ANY that apply:

  • New Tests
  • [ x] Existing Tests
  • Test Framework
  • CI System
  • Other

Consensus Changes

  • Consensus Changes

API Changes

  • API Changes

Documentation Additions

  • Documentation Additions

@praphael praphael requested review from victorj8 and bogniq June 23, 2021 02:10
@nksanthosh
Copy link
Contributor

Related issue #10435

@praphael praphael changed the title fix bug in trace_api_plugin no params returned GH issue 10435 / EPE 1066 fix bug in trace_api_plugin no params returned GH issue 10435 / EPE 1066 https://github.com/EOSIO/eos/issues/10435 Jun 23, 2021
praphael added a commit that referenced this pull request Jun 23, 2021
fix bug in trace_api_plugin no params returned GH issue 10435 / EPE 1066 #10435 #10449
@praphael praphael merged commit fa3bc50 into release/2.1.x Jun 23, 2021
@praphael praphael deleted the praphael/EPE-1066 branch July 8, 2021 15:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants