GET /organisations/{orgID}/users returns password hash and emailHash
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
- typescript
Línea de trabajo
Lee OrganisationsController.ts y localiza getOrganisationUsers, que usa include: { users: true }; revisa las correcciones relacionadas en #878 y #926 para tener contexto. Añade una prueba que compruebe que password y emailHash no están presentes en la respuesta de esta ruta. Después, verifica que la inclusión de usuarios omite ambos campos y ejecuta las pruebas pertinentes.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
GET /organisations/{orgID}/users returns every column of each user in the organisation, including the stored password hash and emailHash, so any OrganisationAdmin can read them.
getOrganisationUsers in OrganisationsController.ts uses include: { users: true } with no omit. #878 and #926 fixed the same leak on the /users routes, but this route was missed. The handler also logs the full response, which #943 already covers.
Fix: include: { users: { omit: { password: true, emailHash: true } } }, plus a test that asserts both fields are absent. If #937 lands first, its UserResponse[] response type leaves both fields out of the type but not out of the response, so the omit is still needed.
Related: #921, #922, #943, #937.
- Lenguaje dominante
- TypeScript
- Estrellas
- 4
- Forks
- 0
- Merge medio
- 14 h 50 min
- PR fusionados (30 d)
- 5
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Tiene una 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 Garvan-Data-Science-Platform/ctrl
-
Dificultad 1/5 1-3 horas Aptitud para principiantes 82/100
-
admin-client backend bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
-
backend bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Garvan-Data-Science-Platform/ctrl#970 · 1 comentario ·
-
backend bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
-
backend bug enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Garvan-Data-Science-Platform/ctrl#956 · 1 comentario ·
Todos los issues de Garvan-Data-Science-Platform/ctrl
Issues similares
-
Add: VIVE TVAbiertoapproved check:passed streams:add
Dificultad 1/5 Menos de una hora Aptitud para principiantes 72/100
iptv-org/iptv#55293 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
bug needs-triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
Fission-AI/OpenSpec#2084 ·
Los mantenedores suelen responder en 2 días
-
Contrib indemnité de départ à la retraite : faire le pont avec le simulateur sur la page génériqueAbiertoP1
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
SocialGouv/code-du-travail-numerique#7561 ·
Los mantenedores suelen responder en 1 día
-
beginner bug
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
hiero-ledger/hiero-sdk-js#4517 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
kubernetes-sigs/headlamp#7950 ·
Los mantenedores suelen responder en 3 días