Merge pull request #347 from berty/dev/moul/always-sign-urls

fix: always sign URLs, even with an empty salt
berty · Jun 15, 2020 · 3308fdb · 3308fdb
2 parents aadaea5 + 573a92b
commit 3308fdb
Showing 1 changed file with 3 additions and 5 deletions.
diff --git a/go/pkg/yolopb/types.go b/go/pkg/yolopb/types.go
@@ -14,11 +14,9 @@ var (
 
 // PrepareOutput adds new fields containing URLs with a signature and filters sensitive/useless data
 func (b *Build) PrepareOutput(salt string) error {
-	if salt != "" {
-		for _, artifact := range b.HasArtifacts {
-			if err := artifact.AddSignedURLs(salt); err != nil {
-				return err
-			}
+	for _, artifact := range b.HasArtifacts {
+		if err := artifact.AddSignedURLs(salt); err != nil {
+			return err
 		}
 	}