Request: make preg_match for content types case insensitive

[clone45]

Hello,

We ran into a little problem with emails that have content types that look like:

Content-type: text/html; charset=utf-8

The regular expression in plancake email parser looks like this:

if (preg_match('/^Content-Type: ?text/html/', $line, $matches)) {

This regular expression won't match because of the capitalization of the "t" in "-type". In other words:

Content-type: << ours
Content-Type: << expected

How about making the preg_match case insensitive?

Cheers, and thanks for the wonderful tool!

• Bret

[plancake]

Hi Bret.

I am glad you are finding the Plancake Email Parser useful.

Thanks a lot for your bug reporting. We hopefully will get sorted out
within the next two weeks.

Thanks,
Dan

On Tue, May 8, 2012 at 7:14 PM, clone45 <
reply@reply.github.com

wrote:

Hello,

We ran into a little problem with emails that have content types that look
like:

Content-type: text/html; charset=utf-8

The regular expression in plancake email parser looks like this:

if (preg_match('/^Content-Type: ?text/html/', $line, $matches)) {

This regular expression won't match because of the capitalization of the
"t" in "-type". In other words:

Content-type: << ours
Content-Type: << expected

How about making the preg_match case insensitive?

Cheers, and thanks for the wonderful tool!

• Bret

---

Reply to this email directly or view it on GitHub:
#5