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

how to limit excess authority when registering a storage node #8594

Open
0xpatrickdev opened this issue Dec 1, 2023 · 2 comments
Open

how to limit excess authority when registering a storage node #8594

0xpatrickdev opened this issue Dec 1, 2023 · 2 comments
Labels
cosmic-swingset package: cosmic-swingset enhancement New feature or request mn2 Issue related to a MN2 dapp needs-design security

Comments

@0xpatrickdev
Copy link
Member

0xpatrickdev commented Dec 1, 2023

What is the Problem Being Solved?

In a permit for a core-eval, a contract can request access to chainStorage like this:

{
  "consume": {
    "chainStorage": true
  }
}

This grants excess authority and the ability to overwrite other storage nodes.

first observed as a potential SDK improvement in #8194 (comment) - @raphdev

Description of the Design

A similar approach to this suggestion, where chainStorage is limited to particular path(s):

{
  "consume": {
    "chainStorage": {
      "myNode": true
    }
  }
}

Security Considerations

The goal of this ticket is to limit excess authority granted around chainStorage during the core-eval process.

Scaling Considerations

Test Plan

Upgrade Considerations

@0xpatrickdev 0xpatrickdev added enhancement New feature or request cosmic-swingset package: cosmic-swingset security needs-design mn2 Issue related to a MN2 dapp labels Dec 1, 2023
@raphdev
Copy link
Contributor

raphdev commented Dec 1, 2023

Worth noting this was first observed as a potential SDK improvement in #8194 (comment)

@dckc
Copy link
Member

dckc commented Jun 20, 2024

We have a prototype in Agoric/dapp-offer-up@0c894f6

I don't think that one (Agoric/dapp-offer-up#61 ) has landed, but the one in dapp-agoric-basics has:

cc @Jovonni

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 mn2 Issue related to a MN2 dapp needs-design security
Projects
None yet
Development

No branches or pull requests

3 participants