Closed jagerman closed 1 year ago
should have done everything in a single review, sorry about that :)
We need more experts to review the design and implementation. I hope @KeeJef will publish more internal design drafts before an actual software release. We have seen examples in the past where in-house developed code was vulnerable to algebraic attacks due to subtle mistakes by the core team. This situation further evidences the need for more external volunteers like @frtget to provide valuable opinions.
Adds a group info config type, which tracks distributed info for v2 groups.
This requires introducing/using a few new concepts not currently used for user config messages:
This doesn't yet add support for sharing and distributing those keys, just for having being able to load a config with a list of multiple possible keys.
This required various modifications to make the config library properly prevent modifications when we can't modify, and to properly follow that through in terms of merging, updates, etc.
TODO: