Skip to content

Commit

Permalink
chore: improve explanatory comment in dynamic contract
Browse files Browse the repository at this point in the history
  • Loading branch information
andreivladbrg committed Nov 28, 2023
1 parent 83dc399 commit 57df8f0
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion src/SablierV2LockupDynamic.sol
Original file line number Diff line number Diff line change
Expand Up @@ -515,7 +515,9 @@ contract SablierV2LockupDynamic is
// Log the cancellation.
emit ISablierV2Lockup.CancelLockupStream(streamId, sender, recipient, asset, senderAmount, recipientAmount);

// Emits an ERC-4906 event to trigger an update of the NFT metadata. We are
// Emits an ERC-4906 event to trigger an update of the NFT metadata. The `updateMetadata` modifier isn't used
// here to avoid DOS attacks by malicious recipient contracts, which could force excessive gas costs on the
// sender.
emit MetadataUpdate({ _tokenId: streamId });

// Interactions: if the recipient is a contract, try to invoke the cancel hook on the recipient without
Expand Down

0 comments on commit 57df8f0

Please sign in to comment.