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

Update wordpress.sh #1923

Closed
wants to merge 1 commit into from
Closed

Update wordpress.sh #1923

wants to merge 1 commit into from

Conversation

vsmash
Copy link

@vsmash vsmash commented Jan 14, 2024

escape $ in location redirect.
I found that the $1 was in the nginx config on vagrant machine. (arm64/parallels)
I escaped the $1 to fix. I also removed the https:// before ${11} (but not in this pr) as i prefer to specify the scheme in the yaml

escape $ in location redirect
@svpernova09
Copy link
Contributor

Thanks for taking the time for this.

prefer to specify the scheme in the yaml

The Homestead.yaml isn't aware of a scheme and shouldn't be.

@vsmash vsmash deleted the nginx-redirect-fix branch January 19, 2024 19:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants