Merge pull request #1360 from snyk/fix/html-to-stdout

Use voidprinter when sending HTML output to stdout

Author: sundowndev

pkg/cmd/scan/output/output.go

@@ -89,6 +89,11 @@ func GetPrinter(config OutputConfig, quiet bool) output.Printer {
 			return &output.VoidPrinter{}
 		}
 		fallthrough
+	case HTMLOutputType:
+		if isStdOut(config.Path) {
+			return &output.VoidPrinter{}
+		}
+		fallthrough
 	case ConsoleOutputType:
 		fallthrough
 	default:

pkg/cmd/scan/output/output_test.go

@@ -529,6 +529,18 @@ func TestGetPrinter(t *testing.T) {
 			key:  PlanOutputType,
 			want: &output.VoidPrinter{},
 		},
+		{
+			name: "html stdout output",
+			path: "stdout",
+			key:  HTMLOutputType,
+			want: &output.VoidPrinter{},
+		},
+		{
+			name: "html /dev/stdout output",
+			path: "/dev/stdout",
+			key:  HTMLOutputType,
+			want: &output.VoidPrinter{},
+		},
 	}
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {