new updates

cmd/root.go

@@ -47,7 +47,7 @@ func Execute() {
 func init() {
 	persistent := rootCmd.PersistentFlags()
 	persistent.StringP("config", "c", "config.toml", "Config file path.")
-	persistent.StringSliceP("reporters", "r", []string{"slack"}, "Specify a list of reporters for reporting vulnerabilities.")
+	persistent.StringSliceP("reporters", "r", []string{"slack", "console"}, "Specify a list of reporters for reporting vulnerabilities.")
 	persistent.BoolP("quiet", "q", false, "Suppress all console output. (Mutually exclusive with 'verbose'.)")
 	persistent.CountP("verbose", "v", "More verbose output. Specifying multiple times increases verbosity. (Mutually exclusive with 'quiet'.)")
 

config/config_test.go

@@ -4,7 +4,6 @@ import (
 	"fmt"
 	"os"
 	"path/filepath"
-	"strings"
 	"testing"
 
 	"github.com/stretchr/testify/assert"
@@ -81,8 +80,7 @@ func TestGetUserConfigFromFile(t *testing.T) {
 }
 
 func TestGetUserConfigFromEnv(t *testing.T) {
-	sliceAsStrings := strings.Join([]string{"slack"}, ",")
-	t.Setenv("VULNBOT_REPORTERS", sliceAsStrings)
+	t.Setenv("VULNBOT_REPORTERS", "slack")
 	t.Setenv("VULNBOT_GITHUB_ORG", "hitchhikers")
 	// This should override the config file
 	t.Setenv("VULNBOT_DEFAULT_SLACK_CHANNEL", "other_slack_channel")
@@ -93,8 +91,7 @@ func TestGetUserConfigFromEnv(t *testing.T) {
 	cfg, err := config.GetUserConfig(testDataPath)
 	assert.Nil(t, err)
 
-	deserializedSlice := strings.Split(sliceAsStrings, ",")
-	assert.Equal(t, deserializedSlice, cfg.Reporters)
+	assert.Equal(t, []string{"slack"}, cfg.Reporters)
 	assert.Equal(t, "hitchhikers", cfg.Github_org)
 	assert.Equal(t, "other_slack_channel", cfg.Default_slack_channel)
 }

internal/scan.go

@@ -45,7 +45,10 @@ func Scan(cmd *cobra.Command, args []string) {
 		}
 	}
 
-	reporters = append(reporters, &reporting.ConsoleReporter{Config: &cfg})
+	if slices.Contains(cfg.Reporters, "console") {
+		reporters = append(reporters, &reporting.ConsoleReporter{Config: &cfg})
+	}
+
 	reportTime := time.Now().UTC()
 	wg := new(sync.WaitGroup)