Skip to content

Commit

Permalink
proto: fix witness request definition
Browse files Browse the repository at this point in the history
  • Loading branch information
hdevalence committed Jan 4, 2024
1 parent 547c2c4 commit 8210c17
Showing 1 changed file with 5 additions and 8 deletions.
13 changes: 5 additions & 8 deletions proto/penumbra/penumbra/view/v1alpha1/view.proto
Original file line number Diff line number Diff line change
Expand Up @@ -90,12 +90,12 @@ service ViewProtocolService {
// Translates a high-level intent ("send X funds to Y address") into a complete transaction plan.
rpc TransactionPlanner(TransactionPlannerRequest) returns (TransactionPlannerResponse);

// Returns authentication paths for the given note commitments.
// Returns authentication data for the given transaction plan.
//
// This method takes a batch of input commitments, rather than just one, so
// that the client can get a consistent set of authentication paths to a
// common root. (Otherwise, if a client made multiple requests, the wallet
// service could have advanced the state commitment tree state between queries).
// This method takes a complete transaction plan, so that the client can get a
// consistent set of authentication paths to a common root for the entire
// transaction. (Otherwise, if a client made multiple requests, the wallet
// service could have advanced the state commitment tree between queries).
rpc Witness(WitnessRequest) returns (WitnessResponse);

// Like `Witness`, but immediately uses the witness data to build (prove) the transaction.
Expand Down Expand Up @@ -357,9 +357,6 @@ message NotesForVotingRequest {
}

message WitnessRequest {
// The note commitments to obtain auth paths for.
repeated crypto.tct.v1alpha1.StateCommitment note_commitments = 2;

// The transaction plan to witness
core.transaction.v1alpha1.TransactionPlan transaction_plan = 3;
}
Expand Down

0 comments on commit 8210c17

Please sign in to comment.