validator: Verify MX record of email addresses #2042

Merged
merged 3 commits into from May 13, 2015

Projects

None yet

2 participants

@greezybacon
Member

This allows detection of some incorrectly-typed email addresses before tickets, users, and agents are created.

It also adds a warning to the ticket view page to alert agents of invalid, in-use email addresses. This may happen for emails processed into tickets when the Reply-To header is used, as well as for old tickets created before this patch.

greezybacon added some commits May 11, 2015
@greezybacon greezybacon validator: Verify MX record of email addresses
This allows detection of some incorrectly-typed email addresses before
tickets, users, and agents are created.
35992da
@greezybacon greezybacon email: Add warning to ticket view page
If the email address in use by the end user is not valid, display a warning
on the ticket view page. This allows an agent to easily flag potential spam
or correct a legitimate email address before sending a response and
receiving a bounce notice.
4bfbd98
@greezybacon greezybacon referenced this pull request May 11, 2015
Closed

1.8.x #2035

@greezybacon greezybacon commented on an outdated diff May 13, 2015
scp/banlist.php
@@ -52,7 +52,7 @@
case 'add':
if(!$filter) {
$errors['err']=sprintf(__('%s: Unknown or invalid'), __('ban list'));
- }elseif(!$_POST['val'] || !Validator::is_email($_POST['val'])) {
+ }elseif(!$_POST['val'] || !Validator::is_valid_email($_POST['val'])) {
@greezybacon
greezybacon May 13, 2015 Member

This should be allowed (test@example.com)

@protich protich merged commit 08b583e into osTicket:develop May 13, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment