Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[Consensys] 4.3 Segregation of duties and permissions. #237

Open
0xdcota opened this issue Mar 8, 2022 · 0 comments
Open

[Consensys] 4.3 Segregation of duties and permissions. #237

0xdcota opened this issue Mar 8, 2022 · 0 comments
Assignees
@0xdcota
Labels
Smart Contracts Solidity related

Comments

@0xdcota
Copy link
Contributor

0xdcota commented Mar 8, 2022

In the FujiERC1155 contract, the onlyPermit modifier should not include owner.

https://github.com/Fujicracy/fuji/blob/bbeb01e0270166eca72ed8dc34649a08d0a5f6ea/packages/hardhat/contracts/abstracts/fujiERC1155/F1155Manager.sol#L34
@0xdcota 0xdcota added the Smart Contracts Solidity related label Mar 8, 2022
@0xdcota 0xdcota self-assigned this Mar 8, 2022
@brozorec brozorec changed the title [Consensys 4.3] Segregation of duties and permissions. [Consensys] 4.3 Segregation of duties and permissions. Mar 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@0xdcota 0xdcota

Labels
Smart Contracts Solidity related
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@0xdcota