Skip to content

Commit

Permalink
No longer include :code and :heappages in execution proofs (paritytec…
Browse files Browse the repository at this point in the history
…h#10419)

* No longer include :code and :heap_pages in execution proofs

* Oops, forgot the line

* Rustfmt
  • Loading branch information
tomaka authored and grishasobol committed Mar 28, 2022
1 parent 3fa78f4 commit cf33aed
Showing 1 changed file with 1 addition and 12 deletions.
13 changes: 1 addition & 12 deletions client/service/src/client/client.rs
Original file line number Diff line number Diff line change
Expand Up @@ -1085,18 +1085,7 @@ where
method: &str,
call_data: &[u8],
) -> sp_blockchain::Result<(Vec<u8>, StorageProof)> {
// Make sure we include the `:code` and `:heap_pages` in the execution proof to be
// backwards compatible.
//
// TODO: Remove when solved: https://github.com/paritytech/substrate/issues/5047
let code_proof = self.read_proof(
id,
&mut [well_known_keys::CODE, well_known_keys::HEAP_PAGES].iter().map(|v| *v),
)?;

self.executor
.prove_execution(id, method, call_data)
.map(|(r, p)| (r, StorageProof::merge(vec![p, code_proof])))
self.executor.prove_execution(id, method, call_data)
}

fn read_proof_collection(
Expand Down

0 comments on commit cf33aed

Please sign in to comment.