Skip to content

Magento 2.2.2 truncanting IPV6 for orders #14823

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

Closed
luismaldonado91 opened this issue Apr 23, 2018 · 2 comments
Closed

Magento 2.2.2 truncanting IPV6 for orders #14823

luismaldonado91 opened this issue Apr 23, 2018 · 2 comments
Labels
Issue: Format is valid Gate 1 Passed. Automatic verification of issue format passed

Comments

@luismaldonado91
Copy link

Preconditions

  1. Magento 2.2.2
  2. PHP 7.0.13

Steps to reproduce

  1. Watch for orders placed from IPv6 clients
  2. If you match the IPv6 from the order to a IPv6 validator, you right away noticed that it's truncated to 32 chars, rendering the IP invalid for the order.

Expected result

  1. Correct IPv6 IP format

Actual result

  1. 32 char truncated IPv6

As for now i've modified the column remote_ip for sales_order and quote tables from VARCHAR(32) to VARCHAR(64). Also i've modified x_forwarded_for from VARCHAR(32) to VARCHAR(64) from table sales_order

I think it's imperative a db schema change for this paramter. For me it's causing that the native signifyd extension dont' post orders to console because of the IPv6 truncation.

@magento-engcom-team magento-engcom-team added the Issue: Format is valid Gate 1 Passed. Automatic verification of issue format passed label Apr 23, 2018
@magento-engcom-team
Copy link
Contributor

Hi @luismaldonado91
This report seems to duplicate #10395
Please continue tracking under that reference.

Thank you for collaboration

@luismaldonado91
Copy link
Author

Thanks for the clarification. I think this is kind of critical and it' has been of your knowledge since August 2017. At least for fraud protection with signifyd is critical.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Issue: Format is valid Gate 1 Passed. Automatic verification of issue format passed
Projects
None yet
Development

No branches or pull requests

2 participants