Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enhancement - API - Aplicación indebida de permisos de grupo sobre tabla incorrecta #177

Merged
merged 15 commits into from
Jul 10, 2024

Conversation

jortilles
Copy link
Collaborator

Descripción del Cambio

En la actualidad SDA aplica los permisos sobre las tablas que se usan en las consultas y sobre todas las que estén relacionadas.

En el issue #175 se pide que los permisos de seguridad se apliquen sólo a las tablas que aparecen en la consulta. Y este PR aplica este cambio.

Issue(s) resuelto(s)

Pruebas a realizar para validar el cambio

Realizar consultas sobre EDA con usuarios no administradores y revisar la correspondencia entre los permisos aplicados y los definidos en las tablas de configuración sda_def_XX

Copy link
Collaborator

@PaulaaSTIC PaulaaSTIC left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No se aplica la seguridad de módulos relacionados.
Si creamos la consulta partiendo desde un módulo al que tenemos acceso total y luego vamos a módulos con permisos de grupo, no se aplica la condición de grupo.

No_seguridad en campos relacionads

Juanjo Ortilles added 3 commits July 3, 2024 11:19
Tablas visibles para el usuario y refactorización de cómo se determina si una tabla es visible o no segregando estrategias benévolas y malevolas
@jortilles jortilles requested a review from PaulaaSTIC July 4, 2024 11:02
@jortilles
Copy link
Collaborator Author

Ya esta corregido @PaulaaSTIC ya puedes volver a validar

@jortilles
Copy link
Collaborator Author

Es necesario volver a ejecutar update model

Copy link
Collaborator

@PaulaaSTIC PaulaaSTIC left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Paneles sin acceso:

Si un usuario no administrador accede a un informe donde el módulo principal de un panel es un módulo al que no tenemos acceso, deberíamos mostrar un mensaje indicando que no se tiene acceso a los datos. Podríamos hacerlo con un mensaje en el panel: "Este panel no está disponible por falta de permisos". Esto debe ocurrir tanto si en el panel solo aparecen campos de este módulo como si aparecen de otros módulos a los que sí tenemos acceso. Si el módulo principal no es visible para nosotros, debe aparecer dicho mensaje, ya que la consulta no se puede cargar correctamente. Actualmente, hay un comportamiento diferente que puede generar confusión.

@PaulaaSTIC PaulaaSTIC self-requested a review July 10, 2024 09:11
@PaulaaSTIC PaulaaSTIC merged commit cd89c35 into reporting Jul 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Incidencia - API - Aplicación indebida de permisos de grupo sobre tabla incorrecta
4 participants