Filter vMCP tools/list with tool:read, as the single-server path does

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#6,925 1 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
65/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
go
Lĩnh vực
backend-api-design, security

Hướng nghiên cứu

Bắt đầu tại pkg/vmcp/core/admission.go, đọc FilterTools và AllowToolCall, sau đó so sánh bước kiểm tra quyền của đường xử lý máy chủ đơn. Cập nhật vMCP để kiểm tra tool:read với cùng phương án dự phòng tool:call; xác minh rằng quyền chỉ có tool:read ảnh hưởng đến tools/list và hành vi của tool:call tuân theo #6903.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

Current behavior

pkg/vmcp/core/admission.go (FilterTools and AllowToolCall) checks tool:call only, so a tool:read-only grant has no effect on vMCP.

Wanted

Use the tool:read check, with the same tool:call fallback as the single-server path.

Depends on #6903 (tool:call implies tool:read).

I found this while reviewing #6903. I'd like to work on it if a maintainer agrees it's worth doing. Could you assign it to me once you've had a look? Thanks.

Ngôn ngữ chính
Go
Star
2.3k
Fork
326
Merge trung bình
1 ngày 53 phút
Pull request đã merge (30 ngày)
155

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của stacklok/toolhive

Tất cả issue của stacklok/toolhive

Issue tương tự

Thêm issue về Go

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.