ickb / proposal

Proposal of iCKB: a NervosDAO liquid staking token
http://ickb.org
MIT License
8 stars 0 forks source link

MetaPoints validation in Limit Order Match #10

Closed XuJiandong closed 3 months ago

XuJiandong commented 3 months ago

This transaction can be simplified below:

CellDeps:
    - Limit Order data cell
    - ...
Inputs:
    - Limit Order cell:
        Data:
            - Amount (16 bytes),
            - MintOrderData variant of OrderData
Outputs:
    - Limit Order cell:
        Data:
            - Amount (16 bytes),
            - MatchOrderData variant of OrderData

How can we ensure that the MetaPoint in data section is properly restricted during the transaction?

phroi commented 3 months ago

Hey @XuJiandong, thank you for publicly expressing your interest in iCKB by reviewing the proposal, I personally appreciate a lot!! 🙏

Limit Order's MetaPoint validation is not documented in the proposal, good catch!!