chore: use proto Getter functions instead of fields (that are potentially nil)

samlaf · samlaf · commit d4a7890fe759 · 2024-10-25T13:11:36.000+01:00
diff --git a/verify/verifier.go b/verify/verifier.go
@@ -78,7 +78,7 @@ func (v *Verifier) VerifyCert(ctx context.Context, cert *Certificate) error {
 	}
 
 	// 3 - verify security parameters
-	batchHeader := cert.Proof().BatchMetadata.BatchHeader
+	batchHeader := cert.Proof().GetBatchMetadata().GetBatchHeader()
 	err = v.verifySecurityParams(cert.ReadBlobHeader(), batchHeader.QuorumNumbers, batchHeader.QuorumSignedPercentages)
 	if err != nil {
 		return fmt.Errorf("failed to verify security parameters: %w", err)

Original file line number	Diff line number	Diff line change
`@@ -78,7 +78,7 @@ func (v Verifier) VerifyCert(ctx context.Context, cert Certificate) error {`
`78`	`78`	`}`
`79`	`79`
`80`	`80`	`// 3 - verify security parameters`
`81`		`- batchHeader := cert.Proof().BatchMetadata.BatchHeader`
	`81`	`+ batchHeader := cert.Proof().GetBatchMetadata().GetBatchHeader()`
`82`	`82`	`err = v.verifySecurityParams(cert.ReadBlobHeader(), batchHeader.QuorumNumbers, batchHeader.QuorumSignedPercentages)`
`83`	`83`	`if err != nil {`
`84`	`84`	`return fmt.Errorf("failed to verify security parameters: %w", err)`