[FEATURE]: provide `call_args` in `call_data`

[roaminro]

Is there an existing issue for this?

• I have searched the existing issues

New feature

As a smart contract developer I would like to be able to retrieve, from the authorize entry point, the arguments that were provded in a contract call.

Currently, authorize_arguments.call only provides contract_id and entry_point and relies on the data field to let smart contracts provide additional data. This means that if a smart contract call check_authority and does not provide the contract call args in data then authorize is not able to verify the contract call arguments.

The idea is to add a new field to call_data and provide the contract call args in this new field:

message call_data {
   bytes contract_id = 1 [(btype) = ADDRESS];
   uint32 entry_point = 2;
   bytes caller = 3;
   bytes data = 4;
   bytes call_args = 5;
}

Anything else?

authorize_arguments message definition

call_data message definition

[koinos-ci]

This issue is stale because it has been open for 30 days with no activity.

[roaminro]

not stale

[koinos-ci]

This issue is stale because it has been open for 30 days with no activity.

[koinos-ci]

This issue is stale because it has been open for 30 days with no activity.

[koinos-ci]

This issue is stale because it has been open for 30 days with no activity.

[mvandeberg]

I suppose this issue comes down to the following question. Should the call data be a requirement or a standard?

Let's consider the following scenarios. What if the calling contract lies or doesn't follow the standard, what should authorize return?

Well, it could just refuse to authorize if the data is not expected.

And if the data is incorrect, what is the worst that could happen? You would authorize a call to a contract that was unintended. But what harm could that do? That contract cannot use that authorization to take tokens from you because the token contracts will also check for authority.

The worst that can happen is that a dumb contract gets your authorization to do dumb contract things limited to the dumb contract's space.

I understand the motivation for the proposal, I do not think it adds any significant value. I am happy to continue the discussion.