gforcada / haproxy_log_analysis

HAProxy log analyzer
https://pypi.org/project/haproxy_log_analysis
GNU General Public License v3.0
89 stars 35 forks source link

make ip_counter use client_ip per default. #21

Closed vixns closed 5 years ago

vixns commented 7 years ago

Allow to be the first of multiple captured headers.

coveralls commented 7 years ago

Coverage Status

Coverage remained the same at 100.0% when pulling 9ce705b90173853c4a266dc6f58196546f43ba45 on vixns:get_ip into ead9e6a759560210bd1f91b970afb5bfb09f81cf on gforcada:master.

gforcada commented 7 years ago

@vixns thanks for the pull request I'm rather busy these days but ping me again if within a week I haven't given any feedback on the pull request. Sorry for the delay.

PatrikNorrgard commented 5 years ago

Please accept this PR, as this is exactly what I need.

gforcada commented 5 years ago

@PatrikNorrgard thanks for bringing this up, I will try to review/merge and push it in the next few days, I'm (again :sweat_smile: ) rather busy right now. Please remind me within a few days/next week if I did not get around to reply.

PatrikNorrgard commented 5 years ago

@gforcada Thanks for replying :) I can also confirm that this works as expected, I did manually download the committed files on top of the normal install.

gforcada commented 5 years ago

@vixns @PatrikNorrgard I made a new release: 3.0.0 with this fix, let me know if it works, and specially if it does not :wink: