Address AI feedback 2

lukaszgryglicki · lukaszgryglicki · commit 372ef3020f1c · 2025-10-09T12:34:53.000+02:00
Signed-off-by: Lukasz Gryglicki <lgryglicki@cncf.io> Assisted by [OpenAI](https://platform.openai.com/) Assisted by [GitHub Copilot](https://github.com/features/copilot)
diff --git a/cla-backend-go/v2/signatures/zip_builder.go b/cla-backend-go/v2/signatures/zip_builder.go
@@ -5,6 +5,7 @@ package signatures
 
 import (
 	"bytes"
+	"errors"
 	"fmt"
 	"net/http"
 	"strings"
@@ -155,6 +156,7 @@ func (z *Zipper) buildPDFZip(claType string, claGroupID string) error {
 	if listErr := <-listErrCh; listErr != nil {
 		if cerr := writer.Close(); cerr != nil {
 			log.Warnf("zip writer close failed: %v", cerr)
+			return errors.Join(listErr, cerr)
 		}
 		return listErr
 	}

Original file line number	Diff line number	Diff line change
`@@ -5,6 +5,7 @@ package signatures`
`5`	`5`
`6`	`6`	`import (`
`7`	`7`	`"bytes"`
	`8`	`+ "errors"`
`8`	`9`	`"fmt"`
`9`	`10`	`"net/http"`
`10`	`11`	`"strings"`
`@@ -155,6 +156,7 @@ func (z *Zipper) buildPDFZip(claType string, claGroupID string) error {`
`155`	`156`	`if listErr := <-listErrCh; listErr != nil {`
`156`	`157`	`if cerr := writer.Close(); cerr != nil {`
`157`	`158`	`log.Warnf("zip writer close failed: %v", cerr)`
	`159`	`+ return errors.Join(listErr, cerr)`
`158`	`160`	`}`
`159`	`161`	`return listErr`
`160`	`162`	`}`