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

Add get_raw_code_and_abi chain api endpoint #4507

Merged
merged 2 commits into from
Jul 13, 2018
Merged

Conversation

spoonincode
Copy link
Contributor

Provides:

  1. Correct binary wasm of a contract (previous get_code endpoint butchered it)
  2. Binary of abi (previously not available)

cleos updated to use get_raw_code_and_abi for the "get code" operation

Provides:
1) Correct binary wasm of a contract (previous get_code endpoint butchered it)
2) Binary of abi (previously not available)

cleos updated to use get_raw_code_and_abi for the "get code" operation
@@ -2509,7 +2534,7 @@ int main( int argc, char** argv ) {

//resolver for ABI serializer to decode actions in proposed transaction in multisig contract
auto resolver = [](const name& code) -> optional<abi_serializer> {
auto result = call(get_code_func, fc::mutable_variant_object("account_name", code.to_string()));
auto result = call(get_abi_func, fc::mutable_variant_object("account_name", code.to_string()));
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does anyone know a good/quick way to test this code path? multisig review

@heifner heifner changed the base branch from release/1.0.x to release/1.1 July 12, 2018 21:45
@heifner heifner merged commit 0f523fd into release/1.1 Jul 13, 2018
@heifner heifner deleted the get_raw_code_and_abi branch July 13, 2018 02:56
@heifner
Copy link
Contributor

heifner commented Jul 13, 2018

brief of changes:

https://developers.eos.io/eosio-nodeos/v1.0.0/reference

CHAIN

(post) get_raw_code_and_abi
Returns contract code and abi in raw form. For code this is wasm. For abi this is packed eosio::chain::abi_def type.

BODY PARAMS

  • account_name string REQUIRED Provide an account name

@heifner heifner mentioned this pull request Jul 13, 2018
14 tasks
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.

2 participants