search

eldy / AWStats

AWStats Log Analyzer project (official sources)
https://www.awstats.org
364 stars 121 forks source link

Column mismatch with GeoIP2 #162

Closed Bllacky closed 3 years ago

Bllacky commented 4 years ago

After struggling a bit I've managed to get AWStats to work with GeoIP2. However, it seems in the new version I have there is a mismatch between columns. Has anyone experienced this?

Here is a printscreen: https://iapk2g.db.files.1drv.com/y4pAiNNCJQIFRp6m_jUqrpf9CABBpVLp_9qqhGYOlARId8zXkT48ccDVmNTbq15cCWgAVADXmAlDDdRNZ_v1wT2vI4i7San9Sh-aI_4kIIYMBbP6qJ30xQ_lZeSJVEGYSFA09UwgbCq8yaRf2mSafvISi4gQbgP1tPsTi9e46K-93zcGy3WhHsmjJ6N7n5tvzxG7p22Y518Sn4MbOuCS39JTxZqpBhuk6oWXfDKmMCrAhA/wrong%20columns.png?psid=1

damien-giry commented 4 years ago

Hi,

I think that I have the same problem (the link to your print screen is not working for me).

Snap1

Bllacky commented 4 years ago

Alright! Yes, I have exactly the same problem.

damien-giry commented 4 years ago

It seems that a "branch merge" delete a line in the file wwwroot/cgi-bin/plugins/geoip2_country.pm :

Snap1

Bllacky commented 4 years ago

I will try the above change as well and see if it works.

LE: I can confirm that the proposed fix at line 145 will fix the issue.

vinixwu commented 4 years ago

Thank you, @damien-giry, your solutioin works. :)

kearva commented 3 years ago

This was fixed by the merge in #178, so this issue can be closed.

Bllacky commented 3 years ago

Fixed as per merge.