Workflow Notifications
The module does not properly protect its settings page. Anyone could make unauthorised changes to the notification settings. They might uncover hidden details or alter how messages are sent.
- Who could do thisAnyone visiting the site. No login needed.
- Does it apply to youAny site using this module.
- Has it been used in attacksNo sign of it.
- How urgentDrupal rates this critical. Cyber Essentials expects a fix within 14 days. Do it this week.
Tell your developerUpdate Workflow Notifications to 3.0.3.
For developers: what the fix changed
The fix removes the insecure `WorkflowNotificationControlHandler` which bypassed access checks for update and delete operations. It also adds `admin_permission = "administer workflow"` to the entity annotations in `WorkflowSmsNotify.php` and `WorkflowNotification.php`, and enforces `_permission: 'administer workflow'` in the routing requirements for both modules.
Also in this release Added an option to allow same state transitions, automated Drupal 12 compatibility fixes (moving hooks to classes), and general code cleanup.
composer.json+1 −2config/schema/workflow_notifications.schema.yml+3 −0modules/workflow_sms_notify/composer.json+1 −3modules/workflow_sms_notify/config/schema/workflow_sms_notify.schema.yml+7 −1modules/workflow_sms_notify/src/Entity/WorkflowSmsNotify.php+8 −6 fixmodules/workflow_sms_notify/src/Form/WorkflowSmsNotificationForm.php+2 −1modules/workflow_sms_notify/src/Hook/WorkflowSmsNotifyHooks.php+70 −0modules/workflow_sms_notify/workflow_sms_notify.info.yml+3 −2modules/workflow_sms_notify/workflow_sms_notify.install+3 −1modules/workflow_sms_notify/workflow_sms_notify.module+8 −35modules/workflow_sms_notify/workflow_sms_notify.routing.yml+9 −6 fixmodules/workflow_sms_notify/workflow_sms_notify.services.yml+5 −0