0022701: Using multiple notification e-mails addresses gives an invalid e-mail address message

ClearFoundation Tracker - ClearOS
View Issue Details

ID	Project	Category	View Status	Date Submitted	Last Update
0022701	ClearOS	app-events - Events System	public	2019-01-08 14:10	2021-11-10 08:43

Reporter	NickH
Assigned To	NickH
Priority	normal	Severity	minor	Reproducibility	always
Status	closed	Resolution	fixed
Platform		OS		OS Version
Product Version	7.6.0 Updates
Target Version	7.6.0 Updates	Fixed in Version	7.5.0 Updates

Summary	0022701: Using multiple notification e-mails addresses gives an invalid e-mail address message
Description	If you set up the Events with multiple addresses in the "Email to Notify" field you get an invalid e-mail address error. I believe it is because the explode function in /usr/clearos/apps/events/libraries/Events.php is leaving a trailing space in the e-mail field. If you trim the "$email" parameter before testing, for validity then it works, so change line 1027 from: if (!preg_match("/^([a-zA-Z0-9])+([a-zA-Z0-9\._-])@([a-zA-Z0-9_-])+([a-zA-Z0-9\._-]+)+$/", $email)) to: if (!preg_match("/^([a-zA-Z0-9])+([a-zA-Z0-9\._-])@([a-zA-Z0-9_-])+([a-zA-Z0-9\._-]+)+$/", trim($email)))
Steps To Reproduce
Additional Information
Tags	No tags attached.
Relationships
Attached Files

Issue History
Date Modified	Username	Field	Change
2019-01-08 14:10	NickH	New Issue
2019-01-08 14:12	NickH	Description Updated	bug_revision_view_page.php?rev_id=2461#r2461
2019-01-09 07:53	NickH	Note Added: 0008731
2019-01-15 03:21	NickH	Description Updated	bug_revision_view_page.php?rev_id=2521#r2521
2019-01-15 03:36	NickH	Assigned To	=> NickH
2019-01-15 03:36	NickH	Status	new => assigned
2019-01-15 03:37	NickH	Note Added: 0008751
2019-02-01 12:27	NickH	Status	assigned => resolved
2019-02-01 12:27	NickH	Fixed in Version	=> 7.5.0 Updates
2019-02-01 12:27	NickH	Resolution	open => fixed
2021-11-10 08:43	NickH	Status	resolved => closed

Notes