keep-starknet-strange / piltover

Starknet Core Contract components in Cairo.
MIT License
26 stars 17 forks source link

[bug] add from_address in func compute_message_hash_sn_to_appc #25

Closed suiwater closed 3 weeks ago

suiwater commented 6 months ago

fn compute_message_hash_sn_to_appc( nonce: felt252, to_address: ContractAddress, selector: felt252, payload: Span ) -> felt252 should be: fn compute_message_hash_sn_to_appc(from_address: ContractAddress, nonce: felt252, to_address: ContractAddress, selector: felt252, payload: Span ) -> felt252 https://github.com/keep-starknet-strange/piltover/blob/2be9d46f00c9c71e2217ab74341f77b09f034c81/src/messaging/hash.cairo#L22 https://github.com/keep-starknet-strange/piltover/blob/2be9d46f00c9c71e2217ab74341f77b09f034c81/src/messaging/hash.cairo#L8 https://github.com/starkware-libs/cairo-lang/blob/caba294d82eeeccc3d86a158adb8ba209bf2d8fc/src/starkware/starknet/solidity/StarknetMessaging.sol#L88 https://github.com/starkware-libs/cairo-lang/blob/caba294d82eeeccc3d86a158adb8ba209bf2d8fc/src/starkware/starknet/solidity/StarknetMessaging.sol#L97 uint256(msg.sender), If this is an error, I will modify this bug.

suiwater commented 6 months ago

@glihm

glihm commented 1 month ago

Hey @suiwater sorry for the too huge delay here.

That's a very good catch. Let's add the new param. If you're still around, happy to assign if you can tackle it.

onlydustapp[bot] commented 1 month ago

Hey @glihm! Thanks for showing interest. We've created an application for you to contribute to Blobstream-Starknet. Go check it out on OnlyDust!

suiwater commented 1 month ago

@glihm Accidentally clicking on assignment. sorry.