@git001 opened this Pull Request on May 28th 2014

add the regex named pattern method and http_version for future use

@mattab commented on May 30th 2014 Member

@git001 the build failed after your change: https://travis-ci.org/piwik/piwik/jobs/26204912

It went from expected <nb_pageviews>50</nb_pageviews> to processed <nb_pageviews>49</nb_pageviews>. So what's happening is that one of the request was not tracked.

Here are the log files used for this test: https://github.com/piwik/piwik/tree/master/tests/resources/access-logs

Maybe it works if you replace .*? by \S+ ?

PS: ticket is http://dev.piwik.org/trac/ticket/5261

@mattab commented on June 2nd 2014 Member

feel free to re-open this pull request when you get a chance to solve the problem, thanks!

This Pull Request was closed on June 2nd 2014
Powered by GitHub Issue Mirror