Refactor verbose flag to be global with log-level support (#329)

* Refactor verbose flag to be global with log-level support

Author: frjcomp

docs/introduction/getting_started.md

@@ -77,6 +77,35 @@ Note: `high-verified` cannot be removed, but verification can be disabled using
 
 When you run Pipeleak for the first time, it generates a `rules.yml` file based on [this repository](https://github.com/mazen160/secrets-patterns-db/blob/master/db/rules-stable.yml). You can customize your scan rules by modifying this file as needed.
 
-### Keybindings
+### Log Levels
 
-In the `scan` mode you can change interactively between log levels by pressing `t`: Trace, `d`: Debug, `i`: Info, `w`: Warn, `e`: Error. Pressing `s` will output the current status.
+Pipeleak supports different log levels to control output verbosity:
+
+* **info** (default) - Standard informational messages
+* **debug** - Detailed diagnostic information
+* **warn** - Warning messages only
+* **error** - Error messages only
+* **trace** - Very detailed trace-level logging
+
+You can set the log level in two ways:
+
+**Using the verbose flag:**
+```bash
+# Short form - sets debug level
+pipeleak gl scan --token glpat-xxx --gitlab https://gitlab.com -v
+
+# Long form with specific level
+pipeleak gl scan --token glpat-xxx --gitlab https://gitlab.com --log-level debug
+pipeleak gl scan --token glpat-xxx --gitlab https://gitlab.com --log-level trace
+pipeleak gl scan --token glpat-xxx --gitlab https://gitlab.com --log-level warn
+pipeleak gl scan --token glpat-xxx --gitlab https://gitlab.com --log-level error
+```
+
+**Interactive keybindings:**
+During a scan, you can change log levels on the fly by pressing:
+* `t` - Trace level
+* `d` - Debug level
+* `i` - Info level
+* `w` - Warn level
+* `e` - Error level
+* `s` - Output current scan status

src/pipeleak/cmd/bitbucket/scan.go

@@ -21,7 +21,6 @@ import (
 type BitBucketScanOptions struct {
 	Email                  string
 	AccessToken            string
-	Verbose                bool
 	ConfidenceFilter       []string
 	MaxScanGoRoutines      int
 	TruffleHogVerification bool
@@ -78,8 +77,6 @@ pipeleak bb scan --token ATATTxxxxxx --email auser@example.com --public --maxPip
 	scanCmd.PersistentFlags().BoolVarP(&options.Public, "public", "p", false, "Scan all public repositories")
 	scanCmd.PersistentFlags().StringVarP(&options.After, "after", "", "", "Filter public repos by a given date in ISO 8601 format: 2025-04-02T15:00:00+02:00 ")
 
-	scanCmd.PersistentFlags().BoolVarP(&options.Verbose, "verbose", "v", false, "Verbose logging")
-
 	return scanCmd
 }
 
@@ -88,7 +85,6 @@ func Scan(cmd *cobra.Command, args []string) {
 		log.Fatal().Msg("When using --token you must also provide --email")
 	}
 
-	logging.SetLogLevel(options.Verbose)
 	go logging.ShortcutListeners(scanStatus)
 
 	runner.InitScanner(options.ConfidenceFilter)

src/pipeleak/cmd/bitbucket/scan_test.go

@@ -65,16 +65,12 @@ func TestNewScanCmd(t *testing.T) {
 	if persistentFlags.Lookup("maxPipelines") == nil {
 		t.Error("Expected 'maxPipelines' persistent flag to exist")
 	}
-	if persistentFlags.Lookup("verbose") == nil {
-		t.Error("Expected 'verbose' persistent flag to exist")
-	}
 }
 
 func TestBitBucketScanOptions(t *testing.T) {
 	opts := BitBucketScanOptions{
 		Email:                  "test@example.com",
 		AccessToken:            "token123",
-		Verbose:                true,
 		ConfidenceFilter:       []string{"high", "medium"},
 		MaxScanGoRoutines:      4,
 		TruffleHogVerification: true,
@@ -94,9 +90,6 @@ func TestBitBucketScanOptions(t *testing.T) {
 	if opts.AccessToken != "token123" {
 		t.Errorf("Expected AccessToken 'token123', got %q", opts.AccessToken)
 	}
-	if !opts.Verbose {
-		t.Error("Expected Verbose to be true")
-	}
 	if len(opts.ConfidenceFilter) != 2 {
 		t.Errorf("Expected 2 confidence filters, got %d", len(opts.ConfidenceFilter))
 	}

src/pipeleak/cmd/devops/devops_test.go

@@ -80,7 +80,4 @@ func TestNewScanCmd(t *testing.T) {
 	if persistentFlags.Lookup("maxBuilds") == nil {
 		t.Error("Expected 'maxBuilds' persistent flag to exist")
 	}
-	if persistentFlags.Lookup("verbose") == nil {
-		t.Error("Expected 'verbose' persistent flag to exist")
-	}
 }

src/pipeleak/cmd/devops/scan.go

@@ -16,7 +16,6 @@ import (
 type DevOpsScanOptions struct {
 	Username               string
 	AccessToken            string
-	Verbose                bool
 	ConfidenceFilter       []string
 	MaxScanGoRoutines      int
 	TruffleHogVerification bool
@@ -78,13 +77,10 @@ pipeleak ad scan --token xxxxxxxxxxx --username auser --artifacts --organization
 	scanCmd.Flags().StringVarP(&options.Project, "project", "p", "", "Project name to scan - can be combined with organization")
 	scanCmd.Flags().StringVarP(&options.DevOpsURL, "devops", "d", "https://dev.azure.com", "Azure DevOps base URL")
 
-	scanCmd.PersistentFlags().BoolVarP(&options.Verbose, "verbose", "v", false, "Verbose logging")
-
 	return scanCmd
 }
 
 func Scan(cmd *cobra.Command, args []string) {
-	logging.SetLogLevel(options.Verbose)
 	go logging.ShortcutListeners(scanStatus)
 
 	runner.InitScanner(options.ConfidenceFilter)

src/pipeleak/cmd/gitea/enum/enum.go

@@ -4,15 +4,13 @@ import (
 	"fmt"
 
 	"code.gitea.io/sdk/gitea"
-	"github.com/CompassSecurity/pipeleak/pkg/logging"
 	"github.com/rs/zerolog/log"
 	"github.com/spf13/cobra"
 )
 
 var (
 	giteaApiToken string
 	giteaUrl      string
-	verbose       bool
 )
 
 func NewEnumCmd() *cobra.Command {
@@ -30,8 +28,6 @@ func NewEnumCmd() *cobra.Command {
 }
 
 func Enum(cmd *cobra.Command, args []string) {
-	logging.SetLogLevel(verbose)
-
 	if err := runEnum(giteaUrl, giteaApiToken); err != nil {
 		log.Fatal().Stack().Err(err).Msg("Enumeration failed")
 	}

src/pipeleak/cmd/gitea/gitea.go

@@ -6,10 +6,6 @@ import (
 	"github.com/spf13/cobra"
 )
 
-var (
-	verbose bool
-)
-
 func NewGiteaRootCmd() *cobra.Command {
 	giteaCmd := &cobra.Command{
 		Use:     "gitea [command]",
@@ -21,7 +17,5 @@ func NewGiteaRootCmd() *cobra.Command {
 	giteaCmd.AddCommand(enum.NewEnumCmd())
 	giteaCmd.AddCommand(scan.NewScanCmd())
 
-	giteaCmd.PersistentFlags().BoolVarP(&verbose, "verbose", "v", false, "Verbose logging")
-
 	return giteaCmd
 }

src/pipeleak/cmd/gitea/gitea_test.go

@@ -27,11 +27,6 @@ func TestNewGiteaRootCmd(t *testing.T) {
 		t.Errorf("Expected GroupID 'Gitea', got %q", cmd.GroupID)
 	}
 
-	flags := cmd.PersistentFlags()
-	if flags.Lookup("verbose") == nil {
-		t.Error("Expected 'verbose' persistent flag to exist")
-	}
-
 	if len(cmd.Commands()) < 2 {
 		t.Errorf("Expected at least 2 subcommands, got %d", len(cmd.Commands()))
 	}

src/pipeleak/cmd/gitea/scan/api.go

@@ -18,7 +18,6 @@ type GiteaScanOptions struct {
 	Token                  string
 	GiteaURL               string
 	Artifacts              bool
-	Verbose                bool
 	ConfidenceFilter       []string
 	MaxScanGoRoutines      int
 	TruffleHogVerification bool

src/pipeleak/cmd/gitea/scan/http_utils_test.go

@@ -479,7 +479,6 @@ func setupTestScanOptions() {
 		Token:                  "test-token",
 		GiteaURL:               "https://gitea.example.com",
 		Artifacts:              false,
-		Verbose:                false,
 		ConfidenceFilter:       []string{},
 		MaxScanGoRoutines:      4,
 		TruffleHogVerification: false,