The reorder-sets action in Craft CMS’s GlobalsController is missing the requireAdmin() check that the adjacent save-set and delete-set actions both enforce. Any authenticated control panel user can POST to /actions/globals/reorder-sets and permanently reorder all global sets in the project config, regardless of whether they have admin access. The reordering is written through to the project config and persists across requests.
Description
GlobalsController exposes three administrative actions for managing global set structure. Two of them gate on admin status; the third does not.
Prerequisites
- A Craft CMS instance with at least two global sets and a non-admin control panel user account.
Impact
A non-admin control panel user can reorder all global sets. While this does not expose or modify content, reordering global sets modifies the project config -- a versioned artifact that is typically committed to source control and deployed across environments. An attacker can create noise in the project config history, trigger config-sync conflicts, or manipulate the display order seen by all editors in the admin panel. The same non-admin user cannot create or delete global sets because those actions correctly enforce requireAdmin().
The
reorder-setsaction in Craft CMS’sGlobalsControlleris missing therequireAdmin()check that the adjacentsave-setanddelete-setactions both enforce. Any authenticated control panel user can POST to/actions/globals/reorder-setsand permanently reorder all global sets in the project config, regardless of whether they have admin access. The reordering is written through to the project config and persists across requests.Description
GlobalsControllerexposes three administrative actions for managing global set structure. Two of them gate on admin status; the third does not.Prerequisites
Impact
A non-admin control panel user can reorder all global sets. While this does not expose or modify content, reordering global sets modifies the project config -- a versioned artifact that is typically committed to source control and deployed across environments. An attacker can create noise in the project config history, trigger config-sync conflicts, or manipulate the display order seen by all editors in the admin panel. The same non-admin user cannot create or delete global sets because those actions correctly enforce
requireAdmin().