modelcontextprotocol / modelcontextprotocol/php-sdk
[Client] Implement OAuth scope handling (WWW-Authenticate, scopes_supported, step-up, retry-limit, omitted)
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- PHP
- Estrellas
- 1.6k
- Forks
- 173
- Merge medio
- 2 d 49 min
- PR fusionados (30 d)
- 23
Descripción
Context
Five baseline scenarios cover scope acquisition strategies the client must follow when constructing authorize/token requests:
auth/scope-from-www-authenticate— parsescope="<list>"fromWWW-Authenticateinsufficient_scope challenges and request those scopes on next authorize.auth/scope-from-scopes-supported— when no challenge hint, fall back toscopes_supportedfrom AS metadata.auth/scope-omitted-when-undefined— when neither source provides scopes, omit thescopeparameter entirely (do not send empty string).auth/scope-step-up— on403/401witherror=insufficient_scope, restart auth flow requesting the augmented scope set.auth/scope-retry-limit— bound step-up retries to one to prevent infinite loops.
Scope
Mcp\Client\Auth\ScopeResolverconsulted byAuthCoordinatorbefore every authorize/token request.WwwAuthenticateParserreturningscope,error,error_description,resource_metadata.- Step-up retry counter held in
AuthCoordinatorstate.
Conformance scenarios unblocked
All 5 scope scenarios above.
Dependencies
Blocked by: #316 (401 plumbing), #318 (AS metadata), #319 (Auth Code flow).
Acceptance
- Unit tests per resolution path.
- Conformance: 5 baseline scenarios pass.
cc @soyuka
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Comienza con Mcp\Client\Auth\ScopeResolver y AuthCoordinator; después, inspecciona WwwAuthenticateParser y los cinco escenarios de conformidad nombrados. Verifica cada ruta de resolución de scopes, incluidos los scopes omitidos, y confirma que los reintentos de step-up para insufficient_scope estén limitados a uno; las pruebas unitarias y los cinco escenarios base deberían pasar.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- php
- Área
- api, authentication
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Tranquilo
- Claridad
- Bastante claro
- Aptitud para principiantes
- 48/100