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

Bug #1523219 reported by Kobi Samoray
30
This bug affects 6 people
Affects Status Importance Assigned to Milestone
octavia
Invalid
Wishlist
Unassigned

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)
Changed in neutron:
assignee: nobody → Kobi Samoray (ksamoray)
Revision history for this message
Henry Gessau (gessau) wrote :

Should this be an RFE?

tags: added: lbaas
Revision history for this message
Brandon Logan (brandon-logan) wrote :

should be an rfe especially if its an api addition

Kobi Samoray (ksamoray)
tags: added: rfe
Henry Gessau (gessau)
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)
Changed in neutron:
status: Confirmed → Triaged
tags: added: rfe-approved
tags: removed: rfe
tags: added: rfe
removed: rfe-approved
Kobi Samoray (ksamoray)
description: updated
summary: - [RFE] Support X-Forwarded-For header in LBaaSv2
+ [RFE] Add support X-Forwarded-For header in LBaaSv2
Revision history for this message
Armando Migliaccio (armando-migliaccio) wrote :

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

Revision history for this message
Armando Migliaccio (armando-migliaccio) wrote :

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
Revision history for this message
Armando Migliaccio (armando-migliaccio) wrote :
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
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on neutron-lbaas (master)

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
Revision history for this message
Gregory Thiemonge (gthiemonge) wrote : auto-abandon-script

Abandoned after re-enabling the Octavia launchpad.

Changed in octavia:
assignee: Kobi Samoray (ksamoray) → nobody
status: In Progress → Invalid
tags: added: auto-abandon
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.