-
Notifications
You must be signed in to change notification settings - Fork 684
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
AutoRelayer: VaaKey -> MessageKey #3297
Conversation
There should likely be a forge test testing message key end-to-end right (maybe just with the key being 'identify token bridge message' or something?) - would likely require updates to the MockGenericRelayer to support that message key type edit: Oh I think I see - you did the updates to mockgenericrelayer by just having it be true always but still, maybe at least one forge test that uses MessageKeys end to end would be ideal! |
bfbc383
to
95f3e38
Compare
4433022
to
e02a624
Compare
@@ -195,7 +225,37 @@ abstract contract WormholeRelayerSend is WormholeRelayerBase, IWormholeRelayerSe | |||
refundChain, | |||
refundAddress, | |||
deliveryProviderAddress, | |||
vaaKeys, | |||
WormholeRelayerSerde.vaaKeyArrayToMessageKeyArray(vaaKeys), // why doesn't this work without the 'WormholeRelayerSerde.' ? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you'd have to write vaaKeys.vaaKeyArrayToMessageKeyArray() if you wanted to do this. Not certain, but worth a shot
@@ -112,7 +114,7 @@ abstract contract WormholeRelayerDelivery is WormholeRelayerBase, IWormholeRelay | |||
} | |||
|
|||
// Revert if the VAAs delivered do not match the descriptions specified in the instruction | |||
checkVaaKeysWithVAAs(instruction.vaaKeys, encodedVMs); | |||
checkMessageKeysWithMessagess(instruction.messageKeys, encodedVMs); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe fix this typo - 'Messages'
function checkVaaKeysWithVAAs( | ||
VaaKey[] memory vaaKeys, | ||
bytes[] memory signedVaas | ||
function checkMessageKeysWithMessagess( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
typo
encodeVaaKeyArray can be removed from WormholeRelayerSerde by the way |
Design doc
https://www.notion.so/wormholefoundation/AR-Supports-CCTP-f1f58f0daacc4f37a67e922244e51205