daniele-occhipinti / php-email-parser

PHP Raw Email Parser
GNU Lesser General Public License v3.0
170 stars 81 forks source link

Request: make preg_match for content types case insensitive #5

Open clone45 opened 12 years ago

clone45 commented 12 years ago

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!

plancake commented 12 years ago

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: https://github.com/plancake/official-library-php-email-parser/issues/5