Skip to content

Security: Cross-course assignment-user relation manipulation changes restricted assignment visibility - #9210

Merged
AngelFQC merged 3 commits into
chamilo:masterfrom
AngelFQC:security/auto-fix-GHSA-3xrc-v2xq-pc5f
Oct 12, 2026
Merged

AngelFQC merged 3 commits into
chamilo:masterfrom
AngelFQC:security/auto-fix-GHSA-3xrc-v2xq-pc5f

Conversation

@AngelFQC

@AngelFQC AngelFQC commented Oct 12, 2026 •

Copy link
Copy Markdown
Member

Adding or removing a user restriction on an assignment now requires edit rights on that assignment itself, not only a teacher role in the course of the request. The collection of these relations is also limited to the assignments of the current course and session.

Notes for the reviewer:

  • The target user of a new relation is still not checked against the course members. This PR does not change it.
  • Covered by CStudentPublicationRelUserSecurityTest; each case fails without the part of the fix it covers. Not tested manually in the UI.

Refs GHSA-3xrc-v2xq-pc5f

@AngelFQC
AngelFQC merged commit 221637e into chamilo:master Oct 12, 2026
2 of 7 checks passed
@AngelFQC
AngelFQC deleted the security/auto-fix-GHSA-3xrc-v2xq-pc5f branch October 12, 2026 23:01
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.

1 participant