-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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
Custom Admin Domain and Internal Redirects #28943
Comments
Hi @pocallaghan. Thank you for your report.
Please make sure that the issue is reproducible on the vanilla Magento instance following Steps to reproduce. To deploy vanilla Magento instance on our environment, please, add a comment to the issue:
For more details, please, review the Magento Contributor Assistant documentation. Please, add a comment to assign the issue:
|
Hi @engcom-Lima. Thank you for working on this issue.
|
Hello @pocallaghan Could you confirm that the result will be the same as when using the CLI? |
That is not the same setting, you're setting the "frontname", not the domain name. |
Hello @pocallaghan |
This is long pending issue #12354 |
@engcom-Lima you're likely to be unable to replicate in what ever environment you're trying it in. Changing the domain name of the admin panel requires DNS entries to be added so that the domain name you enter points at the same location as the main stores domain and potentially updates to the server hosting configuration so that the domain loads the right VirtualHost. @sunel it is indeed the same issue, but the ticket was marked as non-issue with no good reason. It is 100% an issue. |
Hi @engcom-Delta. Thank you for working on this issue.
|
✅ Confirmed by @engcom-Delta Issue Available: @engcom-Delta, You will be automatically unassigned. Contributors/Maintainers can claim this issue to continue. To reclaim and continue work, reassign the ticket to yourself. |
Hi @engcom-Charlie. Thank you for working on this issue.
|
This issue also arises, when the default store domain has a subfolder like |
@magento give me 2.4-develop instance |
Hi @engcom-Alfa. Thank you for your request. I'm working on Magento 2.4-develop instance for you |
Hi @engcom-Alfa, here is your Magento instance. |
Hi @pocallaghan. Thank you for your report. The fix will be available with the upcoming 2.4.1 release. |
The logic that checks if a redirect is "internal" is broken in the admin panel when using a custom admin domain, which causes a wide variety of admin redirects to redirect you to the homepage of the default store.
The issue is due to
Magento\Store\App\Response\Redirect::_isUrlInternal
using the default stores base url. When in the adminhtml area, this method should respect the config optionsadmin/url/use_custom
andadmin/url/custom
.Preconditions (*)
http://admin.magento2dev.loc
and addadmin.magento2dev.loc
tohosts
fileSteps to reproduce (*)
123456789
as the request path and987654321
as the target path, then hit save.123456789
as the request path andxyz
as the target path, then hit save.Expected result (*)
Request Path for Specified Store already exists.
Actual result (*)
Please provide Severity assessment for the Issue as Reporter. This information will help during Confirmation and Issue triage processes.
The text was updated successfully, but these errors were encountered: