Menu

#122 Codestriker not supporting ' in email addresses.

open
nobody
None
5
2011-08-25
2011-08-25
Anonymous
No

Codestriker does not support the use of a ' in an email address.

We had trouble getting Codestriker to allow one of my co-workers, who has an irish surname, to be assigned to and create review topics.

I identified the problem as occuring in Input.pm, with changes needing to be made to the _untaint_email and _untaint_emails functions

Changing the regex in the functions allowed us to fix the problem.

_untaint_email
From - '[\s]*[-_\+\w\.{1,200}(\@[-_\+\w\.]{1,200})?[\s]*'
To - '[\s]*[-_\+\w\.\']{1,200}(\@[-_\+\w\.]{1,200})?[\s]*'

_untaint_emails
From - '[\s]*([-_\+\w\.]{1,200}(\@[-_\+\w\.]{1,200})?[\s,;]*){1,100}[\s]*'
To - '[\s]*([-_\+\w\.\']{1,200}(\@[-_\+\w\.]{1,200})?[\s,;]*){1,100}[\s]*'

Discussion


Log in to post a comment.