From d37ac88456d3eb1f2e5b83caececfd6f6b515182 Mon Sep 17 00:00:00 2001
From: Jared Hancock <jared@osticket.com>
Date: Thu, 1 May 2014 13:02:48 -0500
Subject: [PATCH] banlist: Trim new entries and updates via web form
---
scp/banlist.php | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/scp/banlist.php b/scp/banlist.php
index b517a6f55..b0c54f27f 100644
--- a/scp/banlist.php
+++ b/scp/banlist.php
@@ -36,7 +36,7 @@ if($_POST && !$errors && $filter){
}elseif(!$errors){
$vars=array('w'=>'email',
'h'=>'equal',
- 'v'=>$_POST['val'],
+ 'v'=>trim($_POST['val']),
'filter_id'=>$filter->getId(),
'isactive'=>$_POST['isactive'],
'notes'=>$_POST['notes']);
@@ -52,9 +52,9 @@ if($_POST && !$errors && $filter){
$errors['err']='Unknown or invalid ban list';
}elseif(!$_POST['val'] || !Validator::is_email($_POST['val'])) {
$errors['err']=$errors['val']='Valid email address required';
- }elseif(BanList::includes($_POST['val'])) {
+ }elseif(BanList::includes(trim($_POST['val']))) {
$errors['err']=$errors['val']='Email already in the ban list';
- }elseif($filter->addRule('email','equal',$_POST['val'],array('isactive'=>$_POST['isactive'],'notes'=>$_POST['notes']))) {
+ }elseif($filter->addRule('email','equal',trim($_POST['val']),array('isactive'=>$_POST['isactive'],'notes'=>$_POST['notes']))) {
$msg='Email address added to ban list successfully';
$_REQUEST['a']=null;
//Add filter rule here.
--
GitLab