IntersectMBO / IntersectMBO/mithril

Activate `future_snark` feature by default

Open
#2,814 4 comments 0 reactions 1 assignee Claimed by @damrobi View on GitHub
cryptography :closed_lock_with_key: feature :rocket:
Dominant language
Rust
Stars
154
Forks
51
Avg merge
1d 23h
Merged PRs (30d)
38

Description

## Why

Once the pre-aggregation SNARK primitives are implemented (see #2792), we can activate the `future_snark` feature by default. The impact should only exist in the signer and aggregator which will start creating/registering "extended" key registration including Schnorr verification key. We want to be sure that the activation of the feature has no other impact on the Mithril network and that backward compatibility is fully maintained.

## What
Activate the `future_snark` feature by default and provide full backward compatibility for Mithril networks.

## How

- [ ] Make all bytes encoding upgradable without introducing breaking changes in STM library
- [ ] Activate the `future_snark` feature by default in the STM library
- [ ] Check that extended verification keys are created in the signer
- [ ] Check that extended verification keys are broadcast by the leader aggregator
- [ ] Check that the backward compatibility is fully maintained for the Mithril network
- [ ] Bump the versions of the modified crates to the next minor/major

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.