Merge lp:~vishvananda/nova/lp692317 into lp:~hudson-openstack/nova/trunk
Proposed by
Vish Ishaya
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Vish Ishaya | ||||
Approved revision: | 471 | ||||
Merged at revision: | 484 | ||||
Proposed branch: | lp:~vishvananda/nova/lp692317 | ||||
Merge into: | lp:~hudson-openstack/nova/trunk | ||||
Diff against target: |
61 lines (+16/-3) 2 files modified
nova/api/ec2/__init__.py (+7/-1) nova/api/ec2/metadatarequesthandler.py (+9/-2) |
||||
To merge this branch: | bzr merge lp:~vishvananda/nova/lp692317 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Jesse Andrews (community) | Approve | ||
Ed Leafe (community) | Approve | ||
Devin Carlen (community) | Approve | ||
Jay Pipes (community) | Needs Information | ||
Review via email: mp+44169@code.launchpad.net |
Description of the change
Adds a flag to use the X-Forwarded-For header to find the ip of the remote server. This is needed when you have multiple api servers with a load balancing proxy in front. It is a flag that defaults to False because if you don't have a sanitizing proxy in front, users could masquerade as other ips by passing in the header manually.
To post a comment you must log in.
Should this also be done for the openstack API?