[RFE] Add support X-Forwarded-For header in LBaaSv2

Bug #1523219 reported by Kobi Samoray on 2015-12-06
This bug affects 6 people
Affects Status Importance Assigned to Milestone
In Progress
Kobi Samoray

Bug Description

X-Forwarded-For headers are used by proxies and load balancers to pass on the original client's IP to the server, while NATing the request.
This is very handy for some applications but has some overheads and therefore has to be configurable.
LBaaSv2 API doesn't offer support for enabling XFF header.
Without having an XFF header, the members cannot conclude which IP address originated the NATed request - e.g for auditing purposes.
Changes required are addition of a boolean property to the listener - which will indicate that an XFF header should be appended to the requests.

Kobi Samoray (ksamoray) on 2015-12-06
Changed in neutron:
assignee: nobody → Kobi Samoray (ksamoray)
Henry Gessau (gessau) wrote :

Should this be an RFE?

tags: added: lbaas
Brandon Logan (brandon-logan) wrote :

should be an rfe especially if its an api addition

Kobi Samoray (ksamoray) on 2015-12-07
tags: added: rfe
Henry Gessau (gessau) on 2015-12-07
summary: - No support for X-Forwarded-For header in LBaaSv2
+ [RFE] Support X-Forwarded-For header in LBaaSv2
Changed in neutron:
status: New → Confirmed
importance: Undecided → Wishlist
Doug Wiegley (dougwig) on 2015-12-07
Changed in neutron:
status: Confirmed → Triaged
tags: added: rfe-approved
tags: removed: rfe
tags: added: rfe
removed: rfe-approved
Kobi Samoray (ksamoray) on 2015-12-09
description: updated
summary: - [RFE] Support X-Forwarded-For header in LBaaSv2
+ [RFE] Add support X-Forwarded-For header in LBaaSv2

@Doug: what is your recommendation for feature submission? Spec or no spec? This looks like it may require API changes.

If Kobi has the code lined up and Doug is happy to support this, so am I, because this sounds reasonable enough. Mind you to preserve the old way for the health monitor to work.

tags: added: rfe-approved
removed: rfe
Changed in neutron:
status: Triaged → In Progress
Changed in neutron:
milestone: none → mitaka-2
Changed in neutron:
milestone: mitaka-2 → mitaka-3
Changed in neutron:
status: In Progress → Incomplete
assignee: Kobi Samoray (ksamoray) → nobody
milestone: mitaka-3 → none
Changed in neutron:
assignee: nobody → Kobi Samoray (ksamoray)
status: Incomplete → In Progress

Change abandoned by Armando Migliaccio (<email address hidden>) on branch: master
Review: https://review.openstack.org/255963
Reason: This review is > 4 weeks without comment, and failed Jenkins the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

affects: neutron → octavia
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers