femsci/forgejo
femsci/forgejo
1
0
Fork 0
forked from NYANDEV/forgejo
Code Pull requests Activity

Make sure argsSet verifies string isn't empty too (#4980)

Browse source
This commit is contained in:
Lanre Adelowo 2018-12-27 13:38:38 +01:00 committed by Lauris BH
parent 4a685f8b87
commit a82ba73346
1 changed files with 5 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
cmd/cmd.go
View file

@ -9,6 +9,7 @@ package cmd
import ( import (
"errors" "errors"
"fmt" "fmt"
"strings"
"code.gitea.io/gitea/models" "code.gitea.io/gitea/models"
"code.gitea.io/gitea/modules/setting" "code.gitea.io/gitea/modules/setting"
@ -22,6 +23,10 @@ func argsSet(c *cli.Context, args ...string) error {
if !c.IsSet(a) { if !c.IsSet(a) {
return errors.New(a + " is not set") return errors.New(a + " is not set")
} }
if len(strings.TrimSpace(c.String(a))) == 0 {
return errors.New(a + " is required")
}
} }
return nil return nil
} }