[Registration] `invalid_redirect_uri` / `invalid_client_metadata` surface as `OIDErrorCodeOAuthOther`

Abierto Apto para principiantes
  • ¿Algo no está bien?
#989 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
78/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
objective-c
Área
authentication, mobile-dev

Línea de trabajo

Comienza con +[OIDErrorUtilities OAuthErrorCodeFromString:], el método de asignación indicado en los criterios de aceptación, e inspecciona las constantes existentes de códigos de error de OAuth. Agrega las dos asignaciones de errores de registro y cúbrelas con una prueba unitaria que use una respuesta de registro 400 para ambas cadenas de error. Hecho significa que la finalización recibe OIDErrorCodeOAuthInvalidRedirectURI o OIDErrorCodeOAuthInvalidClientMetadata en OIDOAuthRegistrationErrorDomain, y las pruebas unitarias pasan.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

bug

Acceptance Criteria

  • +[OIDErrorUtilities OAuthErrorCodeFromString:] maps invalid_redirect_uri to OIDErrorCodeOAuthInvalidRedirectURI and invalid_client_metadata to OIDErrorCodeOAuthInvalidClientMetadata.
  • A unit test covers both, via a 400 registration response.

Background
OIDErrorCodeOAuthInvalidRedirectURI (-12) and OIDErrorCodeOAuthInvalidClientMetadata (-13) exist in OIDErrorCodeOAuth, with doc comments pointing at OpenID Connect Dynamic Client Registration 1.0 section 3.3, but nothing ever produces them. The mapping table in OAuthErrorCodeFromString: has the ten RFC 6749 codes and stops there.

Repro steps: point performRegistrationRequest:completion: at a registration endpoint that answers 400 {"error":"invalid_redirect_uri"}, then inspect error.code in the completion.

Expected behaviour: error.code == OIDErrorCodeOAuthInvalidRedirectURI in OIDOAuthRegistrationErrorDomain. Same for invalid_client_metadata -> OIDErrorCodeOAuthInvalidClientMetadata.

Actual behaviour: error.code == OIDErrorCodeOAuthOther, so, callers can't tell the two spec'd registration errors apart from an unknown one.

Found while fixing #987.

Lenguaje dominante
Objective-C
Estrellas
2k
Forks
863
Merge medio
6 d 11 h
PR fusionados (30 d)
5

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de openid/AppAuth-iOS

Todos los issues de openid/AppAuth-iOS

Issues similares

Más issues de Objective-C

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.