Skip to content

Commit

Permalink
Rename v2 structs (#1136)
Browse files Browse the repository at this point in the history
  • Loading branch information
ian-shim authored Jan 23, 2025
1 parent cc009aa commit f8c81b2
Show file tree
Hide file tree
Showing 58 changed files with 800 additions and 763 deletions.
6 changes: 3 additions & 3 deletions api/clients/v2/disperser_client.go
Original file line number Diff line number Diff line change
Expand Up @@ -194,13 +194,13 @@ func (c *disperserClient) DisperseBlob(
if err != nil {
return nil, [32]byte{}, fmt.Errorf("error signing blob request: %w", err)
}
blobHeader.Signature = sig
blobHeaderProto, err := blobHeader.ToProtobuf()
if err != nil {
return nil, [32]byte{}, fmt.Errorf("error converting blob header to protobuf: %w", err)
}
request := &disperser_rpc.DisperseBlobRequest{
Data: data,
Blob: data,
Signature: sig,
BlobHeader: blobHeaderProto,
}

Expand Down Expand Up @@ -265,7 +265,7 @@ func (c *disperserClient) GetBlobCommitment(ctx context.Context, data []byte) (*
}

request := &disperser_rpc.BlobCommitmentRequest{
Data: data,
Blob: data,
}
return c.client.GetBlobCommitment(ctx, request)
}
Expand Down
4 changes: 2 additions & 2 deletions api/clients/v2/verification/blob_verifier.go
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ func (v *BlobVerifier) VerifyBlobV2FromSignedBatch(
// to verify that the described blob actually exists in a valid batch.
signedBatch *disperser.SignedBatch,
// Contains all necessary information about the blob, so that it can be verified.
blobVerificationProof *disperser.BlobVerificationInfo,
blobVerificationProof *disperser.BlobInclusionInfo,
) error {
convertedSignedBatch, err := verifierBindings.ConvertSignedBatch(signedBatch)
if err != nil {
Expand Down Expand Up @@ -82,7 +82,7 @@ func (v *BlobVerifier) VerifyBlobV2(
// The header of the batch that the blob is contained in
batchHeader *commonv2.BatchHeader,
// Contains data pertaining to the blob's inclusion in the batch
blobVerificationProof *disperser.BlobVerificationInfo,
blobVerificationProof *disperser.BlobInclusionInfo,
// Contains data that can be used to verify that the blob actually exists in the claimed batch
nonSignerStakesAndSignature verifierBindings.NonSignerStakesAndSignature,
) error {
Expand Down
19 changes: 10 additions & 9 deletions api/docs/common_v2.html

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions api/docs/common_v2.md

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

89 changes: 48 additions & 41 deletions api/docs/disperser_v2.html

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

39 changes: 20 additions & 19 deletions api/docs/disperser_v2.md

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading

0 comments on commit f8c81b2

Please sign in to comment.