diff options
| author | Michael Muré <batolettre@gmail.com> | 2021-04-17 19:40:01 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-04-17 19:40:01 +0200 |
| commit | 6d1c9346cc5ff892f808a7e3dd3e01291e49a16d (patch) | |
| tree | 9b424181369a67f69502a27186bd266a19a28506 /bug/op_create.go | |
| parent | 62fb09a53cc626ac581f33b466a1cdf14eb6ed89 (diff) | |
| parent | 51a2c85954e77068c6afbb4ca54159086220aefd (diff) | |
| download | git-bug-6d1c9346cc5ff892f808a7e3dd3e01291e49a16d.tar.gz | |
Merge pull request #632 from MichaelMure/data-input-cleanup
make sure every text input is safe and validated
Diffstat (limited to 'bug/op_create.go')
| -rw-r--r-- | bug/op_create.go | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/bug/op_create.go b/bug/op_create.go index 75b60bd8..1d01020c 100644 --- a/bug/op_create.go +++ b/bug/op_create.go @@ -3,7 +3,6 @@ package bug import ( "encoding/json" "fmt" - "strings" "github.com/MichaelMure/git-bug/entity" "github.com/MichaelMure/git-bug/entity/dag" @@ -94,11 +93,8 @@ func (op *CreateOperation) Validate() error { if text.Empty(op.Title) { return fmt.Errorf("title is empty") } - if strings.Contains(op.Title, "\n") { - return fmt.Errorf("title should be a single line") - } - if !text.Safe(op.Title) { - return fmt.Errorf("title is not fully printable") + if !text.SafeOneLine(op.Title) { + return fmt.Errorf("title has unsafe characters") } if !text.Safe(op.Message) { |