Skip to content

Commit

Permalink
Remove dependency on originator node for blockchain envelopes (#224)
Browse files Browse the repository at this point in the history
We allow anyone to publish to the blockchain directly, so there is no need to specify the `publisher_node_id`. The `originator_ns` field on the `UnsignedOriginatorEnvelope` will be derived from the block time.
  • Loading branch information
richardhuaaa authored Oct 18, 2024
1 parent 9f31be4 commit 3b52250
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions proto/xmtpv4/envelopes/envelopes.proto
Original file line number Diff line number Diff line change
Expand Up @@ -39,8 +39,7 @@ message PayerEnvelope {
xmtp.identity.associations.RecoverableEcdsaSignature payer_signature = 2;
}

// For blockchain envelopes, the originator_sid is set by the smart contract,
// but the originator_ns is set by the publishing node
// For blockchain envelopes, these fields are set by the smart contract
message UnsignedOriginatorEnvelope {
uint32 originator_node_id = 1;
uint64 originator_sequence_id = 2;
Expand All @@ -50,8 +49,7 @@ message UnsignedOriginatorEnvelope {

// An alternative to a signature for blockchain payloads
message BlockchainProof {
uint64 block_number = 1;
uint32 publisher_node_id = 2;
bytes transaction_hash = 1;
}

// Signed originator envelope
Expand Down

0 comments on commit 3b52250

Please sign in to comment.