Project

General

Profile

Bug #9795

Notification should never be sent if user has mail enabled

Added by Stephen Benjamin over 4 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Normal
Category:
E-Mail
Target version:
Difficulty:
Triaged:
Bugzilla link:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1202558
Description of problem:
Satellite host summary email triggered when email is disabled in user settings

Version-Release number of selected component (if applicable):
Satellite-6.1.0-RHEL-6-20150311.1

How reproducible:
Always

Steps to Reproduce:
1. UI > User -> My Account -> Mail Preferences -> Uncheck Mail Enabled
2. But subscribe to all 5 different email notifications
Puppet error state - Subscribe
- Satellite promote errata - Subscribe
- Satellite sync errata - Subscribe
- Puppet Summary - Daily
- Satellite host advisory - Daily
3. Run #foreman-rake reports:daily

Actual results:
Email notification triggered for Satellite Host Advisory

Expected results:
If Mail Enabled is unchecked, no email should be triggered

Additional info:


Related issues

Has duplicate Foreman - Bug #8795: Email notifications can clutter the logResolved2014-12-28

Associated revisions

Revision 7ca39b34 (diff)
Added by Stephen Benjamin over 4 years ago

fixes #9795 - check if mail is enabled before sending a notification

Revision dea37b4a (diff)
Added by Stephen Benjamin over 4 years ago

fixes #9795 - check if mail is enabled before sending a notification

(cherry picked from commit 7ca39b3442be76da89c2c52d8da0c2d6afb74af1)

Revision 38f96136 (diff)
Added by Dominic Cleal over 4 years ago

refs #9795 - revert string change

Revision cada5482 (diff)
Added by Stephen Benjamin over 4 years ago

fixes #9795 - check if mail is enabled before sending a notification

(cherry picked from commit 7ca39b3442be76da89c2c52d8da0c2d6afb74af1)

History

#1 Updated by Stephen Benjamin over 4 years ago

  • Project changed from Katello to Foreman
  • Subject changed from Satellite host summary email triggered when email is disabled in user settings to Notification should never be sent if user has mail enabled
  • Category deleted (Web UI)

UserMailNotification should handle it, not the individual notifications

#2 Updated by The Foreman Bot over 4 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/2240 added
  • Pull request deleted ()

#3 Updated by Dominic Cleal over 4 years ago

  • Category set to E-Mail

#4 Updated by Anonymous over 4 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

#5 Updated by Dominic Cleal over 4 years ago

  • Legacy Backlogs Release (now unused) set to 36

#6 Updated by Dominic Cleal about 4 years ago

  • Has duplicate Bug #8795: Email notifications can clutter the log added

Also available in: Atom PDF