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

Allow X-Forwarded-For to have multiple values [2.1 backport] #9653

Merged
merged 1 commit into from
May 17, 2017

Conversation

kassner
Copy link
Contributor

@kassner kassner commented May 16, 2017

Backport of #8409

--
PR #1546 was closed because CLA issues, thus I'm resubmitting the changes.

Preconditions

  1. HAProxy/Nginx offloading SSL and sending traffic to Varnish;
  2. Varnish in front of Apache/Nginx.

Steps to reproduce

  1. Place an order according to the preconditions.

Expected result

  1. Current customer IP address stored into quote and order tables.

Actual result

  1. Stored IP is Varnish/HAProxy IP address relative to the webserver (usually 127.0.0.1 or some internal IP, depends on network structure);

@okorshenko okorshenko self-assigned this May 16, 2017
@okorshenko okorshenko added this to the May 2017 milestone May 16, 2017
@magento-team magento-team merged commit 536209c into magento:2.1-develop May 17, 2017
@magento-team
Copy link
Contributor

@kassner thank you for your contribution to Magento 2 project

@kassner kassner deleted the patch-7 branch June 1, 2023 15:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants