diff --git a/mcp/config/config_get.go b/mcp/config/config_get.go index cc31fc4f..4d484fd8 100644 --- a/mcp/config/config_get.go +++ b/mcp/config/config_get.go @@ -3,6 +3,7 @@ package config import ( "context" "encoding/json" + "fmt" "os" "path/filepath" @@ -24,6 +25,10 @@ func handleNginxConfigGet(ctx context.Context, request mcpgo.CallToolRequest) (* args := request.GetArguments() relativePath := mcp.GetString(args, "relative_path") + if relativePath == "" { + return nil, fmt.Errorf("argument 'relative_path' is required") + } + absPath, err := config.ResolveAbsoluteOrRelativeConfPath(relativePath) if err != nil { return nil, err diff --git a/mcp/config/config_history.go b/mcp/config/config_history.go index 167a6ef1..c940819b 100644 --- a/mcp/config/config_history.go +++ b/mcp/config/config_history.go @@ -3,6 +3,7 @@ package config import ( "context" "encoding/json" + "fmt" "github.com/0xJacky/Nginx-UI/internal/mcp" "github.com/0xJacky/Nginx-UI/query" @@ -21,6 +22,10 @@ func handleNginxConfigHistory(ctx context.Context, request mcpgo.CallToolRequest args := request.GetArguments() filepath := mcp.GetString(args, "filepath") + if filepath == "" { + return nil, fmt.Errorf("argument 'filepath' is required") + } + q := query.ConfigBackup var histories, err = q.Where(q.FilePath.Eq(filepath)).Order(q.ID.Desc()).Find() if err != nil {