kassner / log-parser

PHP Web Server Log Parser Library
Apache License 2.0
339 stars 64 forks source link

Update LogParser.php #11

Closed onnimonni closed 10 years ago

onnimonni commented 10 years ago

Add requestTime as it is in README.md. I tested this on nginx logs.

kassner commented 10 years ago

Onni,

Thanks for the contribution! Unfortunately, Apache and Nginx don't represent %T in the same way (Apache won't add the milisecond part), so be aware of that when using this option.

I've added some tests and removed the - from the Regex, since this value is always populated.

Best regards, Rafael Kassner.

onnimonni commented 10 years ago

I realized that apache won't add millisecond part but the regex should still be working. Thanks for the log-parser.