fix: CLI now returns 1 when signature verification fails in JSON mode

Signed-off-by: Pierre-Henri Symoneaux <[email protected]>
ovh · Nov 18, 2024 · fd78395 · fd78395
1 parent 8ae2bbc
commit fd78395
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/cmd/okms/keys/sign.go b/cmd/okms/keys/sign.go
@@ -87,10 +87,11 @@ SIGNATURE can also be passed from a file or stdin using '-' or '@'. Stdin can ho
 			} else {
 				if valid {
 					fmt.Println("Signature is valid")
-				} else {
-					exit.OnErr(errors.New("Signature invalid"))
 				}
 			}
+			if !valid {
+				exit.OnErr(errors.New("Signature invalid"))
+			}
 		} else {
 			resp := exit.OnErr2(common.Client().GetServiceKey(cmd.Context(), keyId, utils.PtrTo(types.Jwk)))
 			if len(*resp.Keys) == 0 {