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

Isolate BLD issuer / mint in its own contract vat #4578

Closed
dckc opened this issue Feb 17, 2022 · 1 comment · Fixed by #4745
Closed

Isolate BLD issuer / mint in its own contract vat #4578

dckc opened this issue Feb 17, 2022 · 1 comment · Fixed by #4745
Assignees
Labels
cosmic-swingset package: cosmic-swingset enhancement New feature or request
Milestone

Comments

@dckc
Copy link
Member

dckc commented Feb 17, 2022

What is the Problem Being Solved?

The BLD issuer kit is created in the bootstrap vat, which means that the mint, issuer, and all the purses are there too. The bootstrap vat gets access to extremely high authority objects; it should only partition access to those object among other components. Running the BLD mint in the same vat is awkward w.r.t. POLA.

Description of the Design

Create a new contract whose only job is to create an IssuerKit. In fact, the mint could be the creatorFacet and the issuer could be the publicFacet.

Security Considerations

See problem description above.

Test Plan

It's not clear that this should result in any observable differences, so basic unit tests for the new contract, plus all the existing tests, seem to suffice.

cc @dtribble @Chris-Hibbert

@dckc dckc added enhancement New feature or request cosmic-swingset package: cosmic-swingset labels Feb 17, 2022
@dckc dckc self-assigned this Feb 17, 2022
@Chris-Hibbert
Copy link
Contributor

Create a new contract whose only job is to create an IssuerKit. In fact, the mint could be the creatorFacet and the issuer could be the publicFacet.

Huzzah! I like this idea.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cosmic-swingset package: cosmic-swingset enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants