Add required argument validation to config_get and config_history handlers

This commit is contained in:
Cursor Agent
2026-04-04 02:33:24 +00:00
parent b787572192
commit 376f14a479
2 changed files with 10 additions and 0 deletions

View File

@@ -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

View File

@@ -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 {