-
Notifications
You must be signed in to change notification settings - Fork 123
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
GeoIP redirect URL contains both target and current store codes #616
Comments
@bbutkovic @dpotkoc I thought we addressed this with #569. Can you investigate. |
We are looking into this. |
Hello, @mykolaoleksiuk, thank you for reporting this issue with us. Unfortunately, we were unable to reproduce the issue, could you just check a couple of things for us to help us get to the bottom of this?
In the case that the base URLs for de or en stores are not use default or use website, we'd like to ask you to let us know of any overrides to the base URLs that you have applied to store/website config. It could also be helpful to see your GeoIP redirection config. One potential cause for this could be that your Magento 2 instance is configured to a different base URL for either the en or de stores and they resemble something along the lines of |
Hello @bbutkovic, We spent some more time investigating the issue and found out that it was caused by a third-party plugin for Magento\Store\Model\Store::isUseStoreInUrl(). |
@mykolaoleksiuk glad you got it fixed and you're welcome :) Despite the issue not being related to us, I took the opportunity to add some tests for the URL rewrite logic in our Magento module that tests against various different configuration and URL variations. We can close this issue but I'll add the test cases for future reference. cc: @vvuksan |
I enabled Fastly GeoIP on my cloud instance.
I added country to store view mapping.
Add store code to Urls configuration is enabled.
When customer is redirected to a target sotre the URL contains both current and target store codes.
Instead of redirecting from https://somedomain.com/de to https://somedomain.com/en it redirects to https://somedomain.com/en/de
The text was updated successfully, but these errors were encountered: