Firstly: Thanks for your work, Sam. SqWebMail 3.0 adds some great
features.
This is a follow-up to my comments about leading spaces in filter rule
match text.
Because a leading space is always illegal in the maildrop filtering
parser, I believe that should always be escaped. And you have included
that ...
Additionally, because maildrop does regular expression matching, there are
many, many other characters that ought to be escaped - principle of least
surprise. Some of my users have been caught when regexes like 'u.k.'
matched text that they didn't expect.
I'm loathe to suggest removing the advanced regex functionality from the
web interface, though - so my suggestion is to allow both "regex-style"
and "plain text" matching.
Interface-wise, I'd like the current
"Header/Body Starts/Contains/Ends With _______"
system to become the plain text match ... escaping all user-entered
characters.
I'd like to see an additional Condition - "Regular Expression" - added.
This could be done in SqWebMail in a hackish way - if the user chooses
"plain text match" simply escape all of the regex control characters. I
think it might be better done in maildrop, though - either with an
additional matching operator or with an additional flag to the existing
regex system ... that's more difficult because it requires changing both
modules though.
Thoughts?
--
Charlie Watts
[EMAIL PROTECTED]
Frontier Internet
http://www.frontier.net/
