[Client] Extend conformance client.php with scenario context env vars
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức phù hợp với người mới
- 72/100
- Loại issue
- Tính năng
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Ít trao đổi
- Công nghệ
- php
- Lĩnh vực
- authentication, testing-qa
Hướng nghiên cứu
Bắt đầu với tests/Conformance/client.php và kiểm tra conformance runner để xác minh chính xác biến môi trường của context và các key JSON. Đọc các method Client\Builder hiện có, sau đó lần theo các auth scenario được liệt kê để kết nối context của chúng; hoàn tất có nghĩa là file ghi lại các key và các auth scenario nhận được context cần thiết cho việc conformance validation.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Context
The conformance runner ships per-scenario context via --context JSON which it passes to the client command. The current tests/Conformance/client.php only reads MCP_CONFORMANCE_SCENARIO and the server URL — it ignores client_id, client_secret, private_key_pem, signing_algorithm, idp_id_token, idp_issuer, idp_token_endpoint, idp_client_id. Without these, auth scenarios cannot exercise the SDK even after auth flows are implemented.
Scope
- Read context from the env var the runner sets (verify exact name; commonly
MCP_CONFORMANCE_CONTEXTas JSON). - For each scenario, wire context into
Client\Builder:auth/pre-registration→setOAuthClient(client_id, client_secret).auth/client-credentials-basic→setOAuthClient(...)+ forceclient_credentialsgrant.auth/client-credentials-jwt→setOAuthClientWithKey(client_id, private_key_pem, signing_algorithm).auth/cross-app-access-complete-flow→ supply IdP id_token + IdP discovery params.
- Land this before or in parallel with auth flow implementation so each PR can be validated against conformance immediately.
Conformance scenarios unblocked
Prerequisite for all auth scenarios requiring context.
Dependencies
Independent — can land first (harness-only).
Acceptance
tests/Conformance/client.phpupdated.- Documented context keys in the test file's docblock.
- Ngôn ngữ chính
- PHP
- Star
- 1.6k
- Fork
- 173
- Merge trung bình
- 2 ngày 20 giờ
- Pull request đã merge (30 ngày)
- 14
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- 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.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của modelcontextprotocol/php-sdk
-
[Server] Handler type uses bare Closure, hard to decorate RegistryInterface under strict PHPStan Đang mởServer
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 78/100
modelcontextprotocol/php-sdk#468 · 2 bình luận ·
-
needs confirmation needs maintainer action Server
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
modelcontextprotocol/php-sdk#398 · 1 reaction ·
-
enhancement
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
modelcontextprotocol/php-sdk#370 ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 65/100
modelcontextprotocol/php-sdk#513 ·
-
enhancement
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 55/100
modelcontextprotocol/php-sdk#510 · 1 bình luận ·
Tất cả issue của modelcontextprotocol/php-sdk
Issue tương tự
-
ArrayCollection::matching() throws a TypeError for composite expressions over array elements Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
doctrine/collections#554 · 1 reaction ·
-
documentation
Độ khó 2/5 1-2 ngày Mức phù hợp với người mới 72/100
components-web-app/docs#97 ·
-
Độ khó 1/5 1-3 giờ Mức phù hợp với người mới 88/100
WordPress/Documentation-Issue-Tracker#2513 · 2 bình luận ·
-
B2B Suite 2.2.0 defect Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
bagisto/b2b-ecommerce#55 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
octobercms/october#6132 ·