import_logs.py fails to detect log type with multiple IPs in first line about matomo-log-analytics HOT 2 OPEN

Hi Guys,

I've pached my import_logs.py as followed.

My Nginx log format :
log_format vhosts '$host $http_x_forwarded_for - $remote_user [$time_local] "$request" $status $body_bytes_sent "$http_referer" "$http_user_agent"';

My Common log format
_COMMON_LOG_FORMAT = (
'(?P<ip>[0-9]*.?[0-9]*.?[0-9]*.?[0-9]*)[\,\s?\S+]*\s+\S+\s+(?P<userid>\S+)\s+\[(?P<date>.*?)\s+(?P<timezone>.*?)\]\s+'
'"\S+\s+(?P<path>.*?)\s+\S+"\s+(?P<status>\S+)\s+(?P<length>\S+)'

[0-9]*.?[0-9]*.?[0-9]*.?[0-9]* allows me to prevent the "," to be part of the <ip> regex group and avoid the strict ipv4 regex filter
[\,\s?\S+]* allows me to have none OR multiple IPs and to keep the first one to be mached in piwik.

Hope it can be usefull, don't hesitate to tel me if my workaround is not a proper one

from matomo-log-analytics.

@AlroneRhyn Would you mind creating a pull request wih your change, and ideally updating our tests to show what you've improved? Thanks!

from matomo-log-analytics.