Allow to pass multple paths to 'updatekeys'

Signed-off-by: Simon Jürgensmeyer <[email protected]>
getsops · Nov 1, 2023 · 7488de6 · 7488de6
1 parent cd4a9ce
commit 7488de6
Showing 1 changed file with 18 additions and 13 deletions.
diff --git a/cmd/sops/main.go b/cmd/sops/main.go
@@ -502,7 +502,7 @@ func main() {
 		},
 		{
 			Name:      "updatekeys",
-			Usage:     "update the keys of a SOPS file using the config file",
+			Usage:     "update the keys of SOPS files using the config file",
 			ArgsUsage: `file`,
 			Flags: append([]cli.Flag{
 				cli.BoolFlag{
@@ -528,18 +528,23 @@ func main() {
 				if c.NArg() < 1 {
 					return common.NewExitError("Error: no file specified", codes.NoFileSpecified)
 				}
-				err = updatekeys.UpdateKeys(updatekeys.Opts{
-					InputPath:   c.Args()[0],
-					GroupQuorum: c.Int("shamir-secret-sharing-quorum"),
-					KeyServices: keyservices(c),
-					Interactive: !c.Bool("yes"),
-					ConfigPath:  configPath,
-					InputType:   c.String("input-type"),
-				})
-				if cliErr, ok := err.(*cli.ExitError); ok && cliErr != nil {
-					return cliErr
-				} else if err != nil {
-					return common.NewExitError(err, codes.ErrorGeneric)
+				failed := false
+				for _, path := range c.Args() {
+					err := updatekeys.UpdateKeys(updatekeys.Opts{
+						InputPath:   path,
+						GroupQuorum: c.Int("shamir-secret-sharing-quorum"),
+						KeyServices: keyservices(c),
+						Interactive: !c.Bool("yes"),
+						ConfigPath:  configPath,
+						InputType:   c.String("input-type"),
+					})
+					if err != nil {
+						failed = true
+						log.Error(err)
+					}
+				}
+				if failed {
+					return common.NewExitError(fmt.Errorf("failed updating keys"), codes.ErrorGeneric)
 				}
 				return nil
 			},