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

Replace AccessControl by AccessControlDefaultAdminRules #192

Closed
Tracked by #210
diego-G opened this issue Jun 6, 2023 · 0 comments
Closed
Tracked by #210

Replace AccessControl by AccessControlDefaultAdminRules #192

diego-G opened this issue Jun 6, 2023 · 0 comments
Assignees
@rya-sge
Labels
enhancement New feature or request Next release The issue has been merged into dev and will be part of the next release Security

Comments

@diego-G
Copy link
Contributor

diego-G commented Jun 6, 2023

The latest recommendation from OpenZeppelin is to use AccessControlDefaultAdminRules. It basically implements some risk mitigations on top of AccessControl. They seem beneficial for the Standard.
@rya-sge rya-sge added enhancement New feature or request Security labels Jun 6, 2023
@rya-sge rya-sge mentioned this issue Jul 28, 2023
Merged
10 tasks
@rya-sge rya-sge mentioned this issue Aug 31, 2023
Merged
@rya-sge rya-sge self-assigned this Aug 31, 2023
@rya-sge rya-sge added the Next release The issue has been merged into dev and will be part of the next release label Sep 4, 2023
@rya-sge rya-sge closed this as completed Sep 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rya-sge rya-sge

Labels
enhancement New feature or request Next release The issue has been merged into dev and will be part of the next release Security
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@diego-G @rya-sge