Is $RT::RTAddressRegexp case-sensitive?

It looks like the regexp to match email addresses to be ignored (for
the purpose of importing To: and Cc: addresses when creating a ticket
from email), $RT::RTAddressRegexp, is being used in a case-sensitive
comparision. See RT/EmailParser.pm line 611, which is the only use
of it I can find:

$address =~ /$RT::RTAddressRegexp/

Is there a good reason not to have an “i” suffix after the “/”?

Since the hostname part of an email address is definitely case-insensitive,
and mailbox usually are, the comparison ought to be case-insensitive to
catch all legal expressions of the addresses.

I’ve confirmed that an address with an uppercase hostname is not matched
by a lowercase pattern that otherwise catches the desired addresses.

-- Larry

Is there a good reason not to have an “i” suffix after the “/”?

I believe a good reason is that you can simply write:

(?i)your_pattern_here

to match your_pattern_here case insensitively, instead of clobbering
other people who do have case-sensitive mailbox settings. :slight_smile:

Thanks,
/Autrijus/