fix relative paths in benchmark test

Author: sean-breen

internal/config/types.go

@@ -9,7 +9,6 @@ import (
 	"context"
 	"errors"
 	"fmt"
-	"log/slog"
 	"path/filepath"
 	"slices"
 	"strings"
@@ -476,14 +475,11 @@ func isAllowedDir(path string, allowedDirs []string) bool {
 }
 
 func checkDirIsAllowed(path string, allowedDirs []string) bool {
-	slog.Info(filepath.Clean(path))
 	if slices.Contains(allowedDirs, path) {
-		slog.Info("Path is allowed", "path", path)
 		return true
 	}
 
-	if path == "/" { // root directory reached, path is not allowed
-		slog.Info("Path is not allowed")
+	if path == "/" || !strings.HasPrefix(path, "/") { // root directory reached with no match, path is not allowed
 		return false
 	}
 

internal/datasource/config/nginx_config_parser.go

@@ -74,7 +74,7 @@ func NewNginxConfigParser(agentConfig *config.Config) *NginxConfigParser {
 }
 
 func (ncp *NginxConfigParser) Parse(ctx context.Context, instance *mpi.Instance) (*model.NginxConfigContext, error) {
-	configPath := instance.GetInstanceRuntime().GetConfigPath()
+	configPath, _ := filepath.Abs(instance.GetInstanceRuntime().GetConfigPath())
 
 	if !ncp.agentConfig.IsDirectoryAllowed(configPath) {
 		return nil, fmt.Errorf("config path %s is not in allowed directories", configPath)

internal/datasource/config/nginx_config_parser_benchmark_test.go

@@ -35,8 +35,9 @@ func BenchmarkNginxConfigParser_Parse(b *testing.B) {
 	for _, configFilePath := range configFilePaths {
 		func(configFilePath string) {
 			b.Run(configFilePath, func(bb *testing.B) {
+				absPath, _ := filepath.Abs(configFilePath)
 				agentConfig.AllowedDirectories = []string{
-					filepath.Dir(configFilePath),
+					filepath.Dir(absPath),
 				}
 
 				nginxConfigParser := NewNginxConfigParser(