GitHub user wilderrodrigues opened a pull request:
https://github.com/apache/cloudstack/pull/549
CLOUDSTACK-8180 - RouterVM does no longer provide X-ForwardedFor header
with Loadbalancer
Hi @remibergsma and @DaanHoogland
The "option forwardfor" in the haproxy has been fixed.
- Make sure the "option forwardfor" is always available by not removing it
- If keepalived is enabled, then set "no option forceclose"
Could you please have a look at this PR?
The first commit is about formatting the code, can be ignored. The second
commit is the actual change.
Cheers,
Wilder
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/schubergphilis/cloudstack fix/CLOUDSTACK-8180
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/cloudstack/pull/549.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #549
----
commit 4dd000b850d3afa98b66b623aaec6f6e8bdcab01
Author: wilderrodrigues <[email protected]>
Date: 2015-07-01T09:51:06Z
CLOUDSTACK-8180 - RouterVM does no longer provide X-ForwardedFor header
with Loadbalancer
- Formatting code
commit 932f8aee261411da75c549f4e4ddaca77a4714d9
Author: wilderrodrigues <[email protected]>
Date: 2015-07-01T09:52:11Z
CLOUDSTACK-8180 - RouterVM does no longer provide X-ForwardedFor header
with Loadbalancer
- Make sure the "option forwardfor" is always available by not removing
it
- If keepalived is enabled, then set "no option forceclose"
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
