Skip to content

HAMT-35: Security Roles#18

Merged
hudsonhrh merged 17 commits intomainfrom
HAMT-35-security
Mar 6, 2025
Merged

HAMT-35: Security Roles#18
hudsonhrh merged 17 commits intomainfrom
HAMT-35-security

Conversation

@jrkosinski
Copy link
Collaborator

@jrkosinski jrkosinski commented Mar 6, 2025

Added the following security roles:

  • PurchaseTracker.authorizeEscrow SYSTEM_ROLE
  • PurchaseTracker.deauthorizeEscrow SYSTEM_ROLE
  • GovernanceVault.setCommunityVault SYSTEM_ROLE
  • GovernanceVault.distributeRewards SYSTEM_ROLE
  • CommunityVault.distribute SYSTEM_ROLE
  • CommunityVault.withdraw SYSTEM_ROLE
  • GovernanceToken.mint MINTER_ROLE
  • GovernanceToken.burn BURNER_ROLE
  • GovernanceToken.depositFor MINTER_ROLE
  • GovernanceToken.withdrawTo MINTER_ROLE

Also:

  • Reorganized some of the Hats-handling code in DeployHamzaVault and HatsDeployment to avoid repetition
  • Added MINTER_ROLE and BURNER_ROLE
  • Modified tests to give out the appropriate roles

@jrkosinski jrkosinski marked this pull request as ready for review March 6, 2025 07:47
@hudsonhrh hudsonhrh merged commit de893ce into main Mar 6, 2025
1 check passed
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.

2 participants