Skip to content

Add FAQ about restricting methods to specific addresses#745

Open
garwalsh wants to merge 5 commits intomainfrom
garwalsh-faq-update
Open

Add FAQ about restricting methods to specific addresses#745
garwalsh wants to merge 5 commits intomainfrom
garwalsh-faq-update

Conversation

@garwalsh
Copy link
Contributor

No description provided.

@vercel
Copy link

vercel bot commented Nov 21, 2023

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs2 Ready Ready Preview, Comment Mar 9, 2026 11:02pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
07-oracles Ignored Ignored Preview Mar 9, 2026 11:02pm

Request Review

@garwalsh garwalsh marked this pull request as ready for review November 26, 2023 15:08
@garwalsh garwalsh requested a review from a team as a code owner November 26, 2023 15:08
```

- Require a private key as a private input, and check that `privateKey.toPublicKey().assertEquals(this.address)`
- Require a signature as an input.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

by Require a signature as an input. do we mean to use the default value of signature? like https://docs.minaprotocol.com/zkapps/o1js/permissions#default-permissions

or do we want to set explicitly set the input field permission to signature? (or something else entirely?)

I was looking for a doc to link to, and found out that I need to learn more about what this means.

Permissions doc https://docs.minaprotocol.com/zkapps/o1js/permissions

signature: Fields that have their permission set to signature can only be manipulated by account updates that are accompanied and authorized by a valid signature.

Or do we mean something as described in https://docs.minaprotocol.com/zkapps/o1js/interact-with-mina#signing-transactions-and-explicit-account-updates ?

Signature authorization – used to update user accounts. Signatures are verified against the account's public key.

@barriebyron
Copy link
Contributor

@garwalsh for questions like this, let's link to a doc for more learning. See my (many) questions, thanks

Co-authored-by: Barrie Byron <barrie@o1labs.org>
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.

3 participants