Trap requests to the OIDC callback endpoint that are missing the cookie that identifies the session
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 38/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- javascript
- Área
- api, authentication, documentation
Línea de trabajo
Comienza localizando el handler GET /api/authn/oidc/callback y la configuración de OIDC mediante AUTHN_OIDC_REDIRECT_ORIGIN. Rastrea cómo llega la cookie de sesión faltante a la biblioteca OIDC y, a continuación, revisa la documentación de configuración de OIDC. Se considera terminado cuando los callbacks sin cookie reciben una respuesta adecuada que no sea 500 y la documentación enfatiza el uso del mismo dominio que la aplicación Workbench.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
The Issue
Requests sent to the GET /api/authn/oidc/callback endpoint must include the cookie which identifies the server-side session for the user. Currently, requests that are missing this cookie result in a 500 Internal Error response. The server log shows the message:
[ERROR] catch all: Error: did not find expected authorization request details in session, req.session["oidc:localhost"] is undefined
This error occurs because the OIDC library expects to find the server-side session data that had been created during the initial call to start the OIDC log in process (GET /api/authn/oidc/login). The server-side session is identified by a cookie sent with the request and when the cookie is absent in the call the the callback endpoint, the REST API creates a new session, instead of accessing the one from the initial request.
The missing cookie can occur if the server can be accessed using two different domain names, and the client accesses the Workbench app using the first domain name, but the OIDC configuration includes the second domain name in the OIDC configuration (configured using the AUTHN_OIDC_REDIRECT_ORIGIN environment variable). This would result in the browser storing the cookie using the first domain name and sending it with any requests to a host with that domain name. But not sending it in requests to hosts with the second domain name.
The Fix
- The REST API should identify requests to the OIDC callback endpoint that do not include the session identifier and send a suitable response, instead of a 500 Internal Error response
- The documentation for configuring OIDC should be updated to emphasize the importance of using the same domain name as when accessing the Workbench app
- Lenguaje dominante
- JavaScript
- Estrellas
- 56
- Forks
- 18
- Merge medio
- 21 h 14 min
- PR fusionados (30 d)
- 10
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Sin plantilla de pull request
- Leer la 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.
Más de mitre-attack/attack-workbench-rest-api
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
mitre-attack/attack-workbench-rest-api#513 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
mitre-attack/attack-workbench-rest-api#505 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
mitre-attack/attack-workbench-rest-api#499 ·
Los mantenedores suelen responder en 1 día
-
Add ADM validation bypass for server-controlled fields on non-compliant objectsPosiblemente ocupada @seansica la tomó hace 12 días. Abierto
mitre-attack/attack-workbench-rest-api#517 · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
Dificultad 3/5 1-2 días Aptitud para principiantes 74/100
mitre-attack/attack-workbench-rest-api#512 · 1 comentario ·
Los mantenedores suelen responder en 1 día
Todos los issues de mitre-attack/attack-workbench-rest-api
Issues similares
-
bug confirmed issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
open-webui/open-webui#33196 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
`printf "%o"` prints decimalAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
fable-compiler/Fable#5157 ·
Los mantenedores suelen responder en 1 día
-
New tool: PaperBananaPosiblemente ocupada @github-actions la tomó hoy. Abiertosubmission triage
Dificultad 1/5 Menos de una hora Aptitud para principiantes 75/100
kausalflow/tools#243 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
area: apple-music area: build-release bug effort: small priority: high
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
ChristianTabor/Argon#32 ·
Los mantenedores suelen responder en 1 día
-
0️⃣ phase/new
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Los mantenedores suelen responder en 1 día