Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Http hosts #22

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Http hosts #22

wants to merge 4 commits into from

Conversation

vixns
Copy link
Contributor

@vixns vixns commented Jan 5, 2017

Adds a counter, top and filter for http host.

The http host must be the second captured request header.

capture request header X-Forwarded-For len 20
capture request header Host len 30

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling d282524 on vixns:http_hosts into ead9e6a on gforcada:master.

@gforcada
Copy link
Owner

gforcada commented Jan 5, 2017

@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.

@gforcada
Copy link
Owner

gforcada commented Jan 6, 2020

@vixns sorry I rewrote haproxy_log_analysis and your pull request is no longer mergeable. I'm so sorry to not have replied to you at all within almost 2 years! 🙇‍♂️ would you still be interested in getting these changes merged? I can offer myself to rebase your changes on top of the current master, I can create a new pull request if you are fine with it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants