Collect and set Winternitz public keys in aggregator #374
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ceyhunsen
force-pushed
the
ceyhun/aggregator_wpk
branch
from
7edb9f9 to
fbb4a59
Compare
ekrembal
reviewed
Dec 10, 2024
…nternitz_public_keys test.
ekrembal
requested changes
Dec 12, 2024
scripts/schema.sql
Outdated
| @@ -130,4 +130,10 @@ create table if not exists header_chain_proofs ( | |||
| proof bytea | |||
| ); | |||
|
|
|||
| -- Verifier table of Winternitz public keys for every operator and time_tx pair | |||
| create table if not exists winternitz_public_keys ( | |||
| watchtower_id int primary key not null, | |||
There was a problem hiding this comment.
Let's save winternitz per operator_id not watchtower_id so that we don't have to load all winternitz pubkeys when signing
ekrembal
approved these changes
Dec 16, 2024
* mock_macro: Add initial. * mock_macro: Add initialize_database. * actor: Pilot mock_macro usage. * mock_macro: Add needed imports for integration tests. * tests: Use new mock_macro in integration tests. * mock_macro: Add create_actors. * mock_macro: Use create_actors everywhere and delete create_actors_grpc. * mock_macro: Use initialize_database everywhere. * mock_macro: Use create_test_config_with_thread_name in everywhere. * test_utils: Rename mock_macro. * cargo: Delete bin entry of `all_servers`. * test_utils: Remove create_database and drop_database to add them in initialize_database. * test_utils: Use get_postgresql_url in initialize_database. * JsonRPC endpoint removals (#377) * jsonrpc: Initial remove. * operator: Remove old rpc test. * verifier: Convert functions to pub. * tests: Delete old file. * cargo: Remove unused features for jsonrpsee.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Adds collecting and saving Winternitz public keys support for
aggragator::setupRPC endpoint.Linked Issues