Canto-Network / Canto

Layer-1 blockchain powered by Free Public Infrastructure
GNU Lesser General Public License v3.0
83 stars 75 forks source link

feat: add decorator to reject msgCreateClawbackVestingAccount in anteHandler #107

Closed tkkwon1998 closed 1 year ago

tkkwon1998 commented 1 year ago

Description


Author Checklist

All items are required. Please add a note to the item if the item is not applicable and please add links to any relevant follow up issues.

I have...

Reviewers Checklist

All items are required. Please add a note if the item is not applicable and please add your handle next to the items reviewed if you only reviewed selected items.

I have...

dongsam commented 1 year ago

Hi, The currently added antehandler seems to be missing the verification of the nested authz message, so to prevent this, we need to add sdk.MsgTypeURL(&vestingtypes.MsgCreateClawbackVestingAccount{}), to here and here as well

tkkwon1998 commented 1 year ago

Hi, The currently added antehandler seems to be missing the verification of the nested authz message, so to prevent this, we need to add sdk.MsgTypeURL(&vestingtypes.MsgCreateClawbackVestingAccount{}), to here and here as well

Thank you, msg type has been added to the authz limiter now.