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.

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

Meta SNIP-75: security council #93

Open
wants to merge 27 commits into
base: main
Choose a base branch
from

Conversation

Eikix
Copy link
Collaborator

@Eikix Eikix commented Jul 6, 2024

Drafting the meta SNIP for Security Council as part of the Fête du SNIP

SNIPS/snip-75.md Outdated Show resolved Hide resolved
SNIPS/snip-75.md Outdated Show resolved Hide resolved
SNIPS/snip-75.md Outdated Show resolved Hide resolved
@lucadonnoh
Copy link

lucadonnoh commented Jul 17, 2024

For reference, this is the current upgradability structure:

image

Starknet would need to give up the permissions for all Proxy governors, Implementation governors, Verifier governors and bridge owners. Also, as per the Risk rosette framework and the Stages framework, the Exit window and Stage designation will stay at zero until forced transactions are implemented and proving is made permissionless.

@Eikix Eikix changed the title draft meta snip for security Meta SNIP: security council Jul 31, 2024
SNIPS/snip-75.md Outdated Show resolved Hide resolved
@Eikix Eikix marked this pull request as ready for review August 1, 2024 11:44
@Eikix Eikix changed the title Meta SNIP: security council Meta SNIP-75: security council Aug 1, 2024
@Eikix
Copy link
Collaborator Author

Eikix commented Sep 8, 2024

Could we get Starkware's opinion and more clarity on the roadmap, @iliav-starkware? (Can't ping Boaz, though I know he owns this track)

@leo-starkware
Copy link
Collaborator

Hey @Eikix, could you add sections "Rationale" and "Security considerations" to the SNIP, in order to keep a uniform syntax among SNIPs? Let's merge afterwards (after renumbering the SNIP)

@Eikix
Copy link
Collaborator Author

Eikix commented Sep 22, 2024

Hey @Eikix, could you add sections "Rationale" and "Security considerations" to the SNIP, in order to keep a uniform syntax among SNIPs? Let's merge afterwards (after renumbering the SNIP)

before merging, could we get Boaz's input or whoever is in charge of the security council is Starknet?
we could get the views of core devs of Starknet, since Security Council triggers downstream big changes to the L1 contracts of Starknet

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.

5 participants