Skip to content

Fix ERC1155 listing update auth for meta-transactions#59

Open
cinnabarhorse wants to merge 1 commit intomasterfrom
codex/fix-erc1155-meta-tx-sender
Open

Fix ERC1155 listing update auth for meta-transactions#59
cinnabarhorse wants to merge 1 commit intomasterfrom
codex/fix-erc1155-meta-tx-sender

Conversation

@cinnabarhorse
Copy link
Member

Summary

  • fix ERC1155 listing update auth check to use LibMeta.msgSender() instead of raw msg.sender
  • preserves expected behavior for direct calls while restoring compatibility for meta-transactions
  • add a Foundry regression test that simulates the meta-transaction call pattern used by the diamond

Test Plan

  • forge test --match-path test/foundryTests/ERC1155MetaTxSellerCheck.t.sol

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant