Update log level range check for lint pass

Author: PichuChen

internal/logging/logger.go

@@ -92,9 +92,15 @@ func NewLogger() Logger {
 		fmt.Println(err)
 		level = 7
 	}
+	if level > 7 {
+		level = 7
+	}
+	if level < 0 {
+		level = 0
+	}
 	return &logger{
 		output: os.Stderr,
-		level:  uint(level),
+		level:  uint(level), // #nosec G115 - level is constrained to 0-7
 	}
 }
 

internal/logging/logger_test.go

@@ -2,11 +2,11 @@ package logging
 
 import (
 	"bufio"
+	"fmt"
 	"io"
 	"io/ioutil"
 	"log"
 	"os"
-	"strconv"
 	"strings"
 	"testing"
 )
@@ -32,7 +32,8 @@ var settingList []LogSetting = []LogSetting{
 }
 
 func testLoggerLevel(t *testing.T, targetLevel uint) {
-	tempFile := newFile("testing level "+strconv.Itoa(int(targetLevel)), t)
+	testName := fmt.Sprintf("testing level %v", targetLevel)
+	tempFile := newFile(testName, t)
 	testLogger := &logger{
 		output: tempFile,
 		level:  targetLevel,

main.go

@@ -25,6 +25,9 @@ func main() {
 	flag.Parse()
 
 	if _, ok := os.LookupEnv("LOG_LEVEL"); !ok {
+		if *logLevel > 7 {
+			*logLevel = 7
+		}
 		os.Setenv("LOG_LEVEL", strconv.Itoa(int(*logLevel)))
 	}