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
WordPress: Use the secure URL for WP_HOME
#1854
Conversation
Seems reasonable to me. I see this actually ends up getting used as $WP_HOME at https://github.com/drud/ddev/blob/2948e7e5d250abfc85dbed76e5af6804e15a912c/pkg/ddevapp/wordpress.go#L143 |
In #1862 DDEV_PRIMARY_URL and app.GetPrimaryURL() are introduced. I think that will be better to use than just using the secure URL by default. |
#1862 has landed, so this can now use the primary URL, will take a look a bit later. In the meantime, rebased this to current upstream/master. |
I took the liberty of rebasing this and using the app.GetPrimaryURL(). However, it doesn't actually work yet with WP because of the basic problem described in #1736 - we need to get that one fixed before this has any value. |
WP_HOME
WP_HOME
Since DDEV instances default to `https`, it makes sense to keep `WP_HOME` the same.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is ready to go now when tests complete. And #1736 is in progress.
The Problem/Issue/Bug:
Every time I run
ddev config
, DDEV overrides thewp-config-ddev.php
for theWP_HOME
constant, and uses HTTP protocol for that URL.However DDEV already defaults to secure URLs, this setting should match that too.
How this PR Solves The Problem:
Changes
DeployURL
to use a secure URL, so thatWP_HOME
can inherit that too.