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

Transaction pool rejects transaction which sets and immediately uses the verification key #12109

Closed
mitschabaude opened this issue Nov 3, 2022 · 3 comments

Comments

@mitschabaude
Copy link
Member

It should be possible to send a zkapp deployment account update (i.e., setting the verification key) and another account update which already uses proof authorization, in the same transaction.
The tx logic which is embedded in snarkyjs also allows such behaviour.
However, a user reports that the graphQL endpoint returns an error:

Couldn't send zkApp command: (Verification_failed "Missing_verification_key: [B62qmjXoXSKzrMGSncSoAnh9i7LhTRuuTWdDbdAe8Nxbo5DH8X7zdWq]")

This seems to indicate to me that the graphQL endpoint doesn't use the verification key updated in the same transaction.

Here is the failing transaction:
https://app.zenhub.com/files/114581326/913f6a1c-f468-496b-9674-86e5ab64bd96/download

@ghost-not-in-the-shell
Copy link
Contributor

ghost-not-in-the-shell commented Jan 3, 2023

Unfortunately, to my memory (I might be wrong), this is not possible in the current version of the protocol. If you just set the verification key then you need to wait for that account update to be included in the transition frontier and the staged ledger being updated, then you can use that verification key in your account update.

@bkase
Copy link
Member

bkase commented Jan 6, 2023

I’m fixing that here:

#12398

@bkase
Copy link
Member

bkase commented Jan 30, 2023

waiting on #12380

@amc-ie amc-ie closed this as completed Feb 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants