Skip to content

Conversation

@joyqvq
Copy link
Collaborator

@joyqvq joyqvq commented Dec 4, 2025

Description

a scaffolding to agree on the design and high level steps.

Test plan

it runs.

@joyqvq joyqvq force-pushed the joy/aggregator-server branch from 758aa00 to 0e5df2d Compare December 5, 2025 15:59
sui-rpc = "0.1.0"
tonic = "0.14"

axum = { version = "0.8", features = ["macros"] }
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let's use those from the other crates

@@ -0,0 +1,32 @@
# Aggregator Server

Aggregator server for Seal committee mode. It fetches encrypted partial keys from committee member servers, verifies encrypted signatures.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Aggregator server for Seal committee mode. It fetches encrypted partial keys from committee member servers, verifies encrypted signatures.
Aggregator server for a Seal committee. It fetches encrypted partial keys from committee member servers, verifies encrypted signatures.

# Aggregator Server

Aggregator server for Seal committee mode. It fetches encrypted partial keys from committee member servers, verifies encrypted signatures.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

also mention that it uses private api keys for accessing the key servers


let app = get_mysten_service::<AppState>(package_name!(), package_version!())
.merge(Router::new().route("/v1/fetch_key", post(handle_fetch_key)))
.with_state(state)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let's also have /service return error

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants