[Client] Add OAuth 2.0 credential storage interface (TokenStorageInterface)
维护者通常 1 天内回复
还没有人认领这个 Issue。
评估
- 难度
- 4/5
- 预计耗时
- 3-5 天
- 新手友好度
- 48/100
- Issue 类型
- 功能
- 描述清晰度
- 基本清楚
- 活跃度
- 冷清
- 技术栈
- php
- 领域
- api, authentication
调研方向
从 Client\Builder 和 auth/* 的 baseline 场景开始，然后检查 src/Client/Auth/ 区域。根据列出的操作和验收标准，跟踪 interface、TokenSet、两个 adapter、builder wiring 以及单元测试；当可选的 builder 配置及其文档都覆盖到时即完成。
由索引模型根据 Issue 内容生成。
描述
Context
Foundation for every OAuth flow on the client side. Without a storage abstraction we cannot persist access tokens, refresh tokens, registered client credentials, or PKCE verifiers across requests/sessions.
Scope
- Define
Mcp\Client\Auth\TokenStorageInterfacewith operations:getToken(string $resourceUrl): ?TokenSet,storeToken(string $resourceUrl, TokenSet $token): void,clearToken(string $resourceUrl): void, plus equivalents for registered client metadata. - Value object
TokenSet(access_token, refresh_token, expires_at, scope, token_type). - Default implementations:
InMemoryTokenStorage(ephemeral, per-process).Psr16TokenStorageadapter for PSR-16CacheInterface.
- Wire into
Client\Builder::setTokenStorage(...)(optional; defaults to in-memory).
Conformance scenarios unblocked
Prerequisite for all auth/* baseline scenarios.
Dependencies
None — must land first.
Acceptance
- Interface + value object + 2 storage adapters in
src/Client/Auth/. - Unit tests for both adapters.
- Builder method documented.
cc @soyuka
- 主要语言
- PHP
- 星标
- 1.6k
- 派生
- 173
- 平均合并
- 19 小时 19 分钟
- 30 天内合并 PR
- 8
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 没有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
modelcontextprotocol/php-sdk 的其他 Issue
-
[Server] Connections to Github Copilot CLI time out on subscriptions/listen可能已有人在做 @SammyTourani 于 4 天前认领。 未关闭bug
难度 2/5 1-3 小时 新手友好度 78/100
modelcontextprotocol/php-sdk#516 ·
维护者通常 1 天内回复
-
[Server] Handler type uses bare Closure, hard to decorate RegistryInterface under strict PHPStan未关闭Server
难度 1/5 1 小时以内 新手友好度 78/100
modelcontextprotocol/php-sdk#468 · 2 条评论 ·
维护者通常 1 天内回复
-
Builder::build() silently skips configured file-based discovery when symfony/finder is missing — should fail loudly可能已有人在做 @ousamabenyounes 于 47 天前认领。 未关闭needs confirmation needs maintainer action Server
难度 2/5 1-3 小时 新手友好度 68/100
modelcontextprotocol/php-sdk#398 · 1 个 reaction ·
维护者通常 1 天内回复
-
enhancement
难度 2/5 1-3 小时 新手友好度 68/100
modelcontextprotocol/php-sdk#370 ·
维护者通常 1 天内回复
-
难度 3/5 1-2 天 新手友好度 74/100
modelcontextprotocol/php-sdk#524 ·
维护者通常 1 天内回复
查看 modelcontextprotocol/php-sdk 的全部 Issue
相似的 Issue
-
Security SecurityBundle
难度 2/5 1-3 小时 新手友好度 68/100
symfony/symfony-docs#23173 ·
维护者通常 3 天内回复
-
难度 2/5 1-3 小时 新手友好度 82/100
fossology/fossology#3893 · 1 条评论 ·
维护者通常 2 天内回复
-
Documentation Feature: Self-Register / Verify UI
难度 2/5 1-3 小时 新手友好度 72/100
维护者通常 1 天内回复
-
i18n
难度 2/5 1-3 小时 新手友好度 74/100
WordPress/wordpress.org#1002 ·
维护者通常 4 天内回复
-
Bug
难度 1/5 1 小时以内 新手友好度 90/100
维护者通常 1 天内回复