pkg/report: fix IsJSON() #2

The PR containers#1226 was merged to soon, it breaks podman tests and backwards compat. `{{json}}` is not a valid template but it worked before the same as `json` so we should keep that. Fixes up commit 152c840 Signed-off-by: Paul Holzinger <[email protected]>
Luap99 · Nov 8, 2022 · 3f47eba · 3f47eba
1 parent bf9e18b
commit 3f47eba
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 4 deletions.
diff --git a/pkg/report/validate.go b/pkg/report/validate.go
@@ -1,12 +1,18 @@
 package report
 
-import "strings"
+import (
+	"regexp"
+)
+
+// check for json and {{json }} which is not valid go template,
+// {{json .}} is valid and thus not matched
+var jsonRegex = regexp.MustCompile(`^\s*(json|{{\s*json\s*}})\s*$`)
 
 // JSONFormat test CLI --format string to be a JSON request
 //
 //	if report.IsJSON(cmd.Flag("format").Value.String()) {
 //	  ... process JSON and output
 //	}
 func IsJSON(s string) bool {
-	return strings.TrimSpace(s) == "json"
+	return jsonRegex.MatchString(s)
 }
diff --git a/pkg/report/validate_test.go b/pkg/report/validate_test.go
@@ -17,8 +17,8 @@ func TestIsJSON(t *testing.T) {
 		{" json", true},
 		{" json ", true},
 		{"  json   ", true},
-		{"{{json}}", false},
-		{"{{json }}", false},
+		{"{{json}}", true},
+		{"{{json }}", true},
 		{"{{json .}}", false},
 		{"{{ json .}}", false},
 		{"{{ json . }}", false},