Doron Fediuck has submitted this change and it was merged. Change subject: tools: notifier - add validation to mail type properties ...................................................................... tools: notifier - add validation to mail type properties Creating smtp requests with FROM/USER properties set to values not in the format user@domain will fail, so we now enforce it the the bash script *?@?* means that there must be a @ with at least one char on each side Change-Id: I652d6b7aa9ca34192d6892174bf57b11335cb98a Bug-Url: https://bugzilla.redhat.com/885436 Signed-off-by: Noam Slomianko <nslomian@redhat.com> --- M backend/manager/tools/src/main/shell/engine-notifier.sh 1 file changed, 9 insertions(+), 0 deletions(-) Approvals: Doron Fediuck: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10775 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I652d6b7aa9ca34192d6892174bf57b11335cb98a Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Noam Slomianko <nslomian@redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck@redhat.com> Gerrit-Reviewer: Noam Slomianko <nslomian@redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs@redhat.com> Gerrit-Reviewer: ofri masad <omasad@redhat.com>