Actions
Bug #20017
closedMail notifications not being sent
Difficulty:
Triaged:
Bugzilla link:
Pull request:
Description
rake reports:daily task fails with following trace
rake aborted! NoMethodError: undefined method `mail_enabled?' for nil:NilClass /home/ares/Projekty/Zdrojaky/foreman/app/models/user_mail_notification.rb:10:in `deliver' /home/ares/Projekty/Zdrojaky/foreman/lib/tasks/reports.rake:97:in `block in process_notifications' /home/ares/Projekty/Zdrojaky/foreman/lib/tasks/reports.rake:96:in `process_notifications' /home/ares/Projekty/Zdrojaky/foreman/lib/tasks/reports.rake:102:in `block (2 levels) in <top (required)>' /home/ares/.rbenv/versions/2.2.2/bin/bundle:23:in `load' /home/ares/.rbenv/versions/2.2.2/bin/bundle:23:in `<main>' Tasks: TOP => reports:daily (See full trace by running task with --trace)
it's obviously caused by taxonomies, the rake task is not running under admin user so mail notification user can not be found if the user is in any context. I think that's a candidate for 1.15.z since it's a regression caused by #16982
Updated by Marek Hulán over 7 years ago
- Related to Bug #16982: CVE-2016-7078 - User with no organizations or locations can see all resources added
Updated by Marek Hulán over 7 years ago
- Related to Tracker #10022: Taxonomies related issues added
Updated by The Foreman Bot over 7 years ago
- Status changed from New to Ready For Testing
- Assignee set to Marek Hulán
- Pull request https://github.com/theforeman/foreman/pull/4589 added
Updated by Marek Hulán over 7 years ago
- Translation missing: en.field_release set to 266
I propose this for 1.15.2, there's no github label yet so just setting here for now.
Updated by Marek Hulán over 7 years ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset cd68b2083022f7c50f45b73f64c3b991613448dd.
Actions