fix lint issue

Author: sean-breen

internal/config/types.go

@@ -504,16 +504,14 @@ func isAllowedDir(path string, allowedDirs []string) (bool, error) {
 		path = filepath.Dir(path)
 	}
 	if err != nil {
-		if os.IsNotExist(err) {
-			// Path does not exist, we can still check if it's an allowed directory
-		} else {
+		if !os.IsNotExist(err) {
 			// Some other error occurred
 			return false, fmt.Errorf("PathError %s %w", path, err)
-		}
+		} // else the path does not exist, we will check if it's in an allowed directory
 	}
 
 	for _, dir := range allowedDirs {
-		// Check if the path is a subdirectory of the allowed directory
+		// Check if the path is a direct match, or is a subdirectory of the allowed directory
 		if slices.Contains(allowedDirs, path) || strings.HasPrefix(path, dir+"/") {
 			slog.Info(fmt.Sprintf("Allowed directory %s is allowed", path))
 			return true, nil

internal/config/types_test.go

@@ -67,15 +67,7 @@ func TestTypes_isAllowedDir(t *testing.T) {
 			filePath: "/not-nginx-test/idontexist.conf",
 		},
 		{
-			name:    "Test 7: File is in allowed directory with trailing slash",
-			allowed: true,
-			allowedDirs: []string{
-				"/etc/nginx/",
-			},
-			filePath: "/etc/nginx/nginx.conf",
-		},
-		{
-			name:    "Test 8: Prefix match",
+			name:    "Test 7: Prefix match not allowed",
 			allowed: false,
 			allowedDirs: []string{
 				"/etc/nginx",