search

UNIMOODLE / moodle-mod_certifygen

GNU General Public License v3.0
1 stars 1 forks source link

Contexto erróneo en verificación de permisos: revokecertificate #37

Closed juacas closed 2 months ago

juacas commented 2 months ago

La comprobación de permiso se hace sobre el curso en lugar de sobre la actividad certifygen:

https://github.com/UNIMOODLE/p31_mod/blob/246fd2e732defd72c4b55990360a58878cad8df0/classes/external/revokecertificate_external.php#L99

Esto puede hacer que alguien con permisos retirados en la actividad ejecute las acciones, entre otros casos adversos.

Los permisos de acciones en un módulo deben checkearse con el contexto de módulo.

elena3ip commented 2 months ago

Esa parte es así por definición del tool_certificate, al ser otro plugin yo no puedo modificarle el contexto, es el que tiene definido.