Project

General

Profile

Actions

Bug #19147

closed

Email notification should be disabled when no host server is configured

Added by Elaad FURREEDAN over 3 years ago. Updated over 3 years ago.

Status:
Released
Priority:
N/A
Target version:
Severity:
Major - prevents use of part of Rudder | no simple workaround
UX impact:
User visibility:
Operational - other Techniques | Rudder settings | Plugins
Effort required:
Priority:
51
Name check:
To do
Fix check:
To do
Regression:

Description

When a change-request is created, the plugin try to send an email even if no hostname is configured for the host server, leading on an error in the webapp.
ERROR change-validation - Error when trying to send email for change request status update from 'changeRequestCreated' to 'Pending validation' <- An error occured. Cause was: AddressException: Illegal address

We should check if there is a hostname configured if so try to send a email notification otherwise do not try to send the email.

Actions #1

Updated by Elaad FURREEDAN over 3 years ago

  • Status changed from New to In progress
Actions #2

Updated by Elaad FURREEDAN over 3 years ago

  • Severity set to Major - prevents use of part of Rudder | no simple workaround
  • User visibility set to Operational - other Techniques | Rudder settings | Plugins
  • Priority changed from 0 to 52
Actions #3

Updated by Elaad FURREEDAN over 3 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Elaad FURREEDAN to François ARMAND
  • Pull Request set to https://github.com/Normation/rudder-plugins/pull/360
Actions #4

Updated by Elaad FURREEDAN over 3 years ago

  • Assignee changed from François ARMAND to Vincent MEMBRÉ
  • Pull Request changed from https://github.com/Normation/rudder-plugins/pull/360 to https://github.com/Normation/rudder-plugins/pull/364
Actions #5

Updated by Elaad FURREEDAN over 3 years ago

  • Assignee changed from Vincent MEMBRÉ to François ARMAND
Actions #6

Updated by Anonymous over 3 years ago

  • Status changed from Pending technical review to Pending release
  • Priority changed from 52 to 51
Actions #7

Updated by Vincent MEMBRÉ over 3 years ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 6.1-1.10 which was released today.

Actions

Also available in: Atom PDF