Plugin: the run_operation RPC does not skip signature checks for non-manager operations
When the run_operation RPC is called on a non-manager operation that has a signature, this signature is checked. This contradicts the description of the RPC, which states that signature checks are skipped.
Since the RPC does not support consensus operations, this affects the voting operations: proposals and ballot.
This issue should be solved once:
-
run_operationis refactored to directly callValidate_operation.validate_operation, which will have a newshould_check_signatureargument (done in !5770 (merged)) -
validate_operationis implemented for voting operations (done in !5828 (merged))
Reminder:
-
Once this issue is fixed, the parameter sign_correctlycan be removed intezt/lib_tezos/operation_core.ml/make_run_operation_input(also done in !5828 (merged)).
Edited by Diane Gallois-Wong