-
Notifications
You must be signed in to change notification settings - Fork 38
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
shared secret #237
shared secret #237
Conversation
@@ -2,7 +2,7 @@ | |||
|
|||
// Q: how are we merging mapping key and ownerPK in edge case? | |||
// Q: should we reduce constraints a mapping's commitment's preimage by not having the extra inner hash? Not at the moment, because it adds complexity to transpilation. | |||
import fs from 'fs'; | |||
import fs, { stat } from 'fs'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
{stat} where stat is used ?
// For a state variable, we'll have passed in `${x}_oldCommitment_value` as a parameter. But our AST nodes will be using `${x}`. This line resolves the two. | ||
if (reinitialisable) | ||
return [ `${typeName ? typeName : 'field'} ${x} = 0`]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
${x}=0, because commitment are not nullified ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is the containsAccessedOnlyStates relevant to this pull request?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is line 63 one of the nullifier changes?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are the changes on line 110 and lines 139-141 related to this pull request or are they from when the nullifier checking was being changed?
This PR adds a decorator
sharedSecret
, and the corresponding api to get the shared keys. If a state is marked as sharedSecret implies that can be state variable is shared between two parties, thus can be nullified by both of them.