cmd: `fmt` exit successfully after overwriting config file (#5351)

Fixes https://github.com/caddyserver/caddy/issues/5349

Amis Shokoohi committed Jan 31, 2023 at 19:54 UTC 8c0b49bf039bc0b6ab72b44ef0e1e6587c76ad2b

1 file changed +4 -1

cmd/commandfuncs.go

+4 -1

@@ -558,7 +558,10 @@ func cmdFmt(fl Flags) (int, error) {

558 if err := os.WriteFile(formatCmdConfigFile, output, 0600); err != nil {

559 return caddy.ExitCodeFailedStartup, fmt.Errorf("overwriting formatted file: %v", err)

560 }

561 - } else if fl.Bool("diff") {

561 + return caddy.ExitCodeSuccess, nil

562 + }

563 +

564 + if fl.Bool("diff") {

565 diff := difflib.Diff(

566 strings.Split(string(input), "\n"),

567 strings.Split(string(output), "\n"))