-
Notifications
You must be signed in to change notification settings - Fork 48
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
Training wheels #613
Comments
One of the other main benefits is that we get to "double-check" the proofs and so any potential soundness bugs in the proving system would not cause catastrophic failures. |
@igamigo wdyt of this as a starting issue? It touches RPC, storage and will require some integration with the client. Might be a bit broad, but there is also no rush on this as users are unlikely to care :) Could also be implemented in stages e.g. (a) update RPC protos & update client, (b) verify via re-execution, (c) figure out long-term storage. |
I agree that this sounds quite broad but looks like good one. After a brief chat with Bobbin we settled on tackling #673 first (while he also moves forward with #174). |
I think from the client/RPC standpoint the main thing that needs to change is that all transactions submitted via
|
We want to (short term) loosen miden's privacy guarantees while we are still actively developing in a centralized fashion. Effectively, miden would have privacy only in the network sense and not from the node operator.
Private transactions will include the transaction data when submitting to the node operator. This data will not appear on-chain (same as it is now), but would be kept separately by the node.
This would act as training wheels, and the intention is to drop this sometime before decentralization.
There are some benefits:
and some downsides:
ito implementation, it makes sense to store this data completely separately from the current database. This would minimize any actual node IO with the data beyond writing it.
This will of course require changes to RPC and the client.
The text was updated successfully, but these errors were encountered: