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

Use the internal URL for nginx proxy pass #4

Merged
merged 1 commit into from Mar 31, 2016

Conversation

MattHall
Copy link
Contributor

ghost_config_url contains the external URL for the site. It's used in
the ghost config.js to set the site URL.

By default, the value of ghost_config_url is set to the internal ghost
URL (i.e. 127.0.0.1:2368). If the user overrides that setting to set it
to the public domain name for the blog (e.g. http://codebeef.com), while
maintaining the value of ghost_config_server, then the blog should be
configures with the correct domain name, and the ghost instance should
still be set to bind to 127.0.0.1:2368, with this value set in the
proxy_pass for nginx.

Currently, changing the ghost_config_url also changes the proxy_pass
setting. This commit changes that so that the proxy_pass setting comes
from the ghost_config_server values.

`ghost_config_url` contains the external URL for the site. It's used in
the ghost config.js to set the site URL.

By default, the value of `ghost_config_url` is set to the internal ghost
URL (i.e. 127.0.0.1:2368). If the user overrides that setting to set it
to the public domain name for the blog (e.g. http://codebeef.com), while
maintaining the value of `ghost_config_server`, then the blog should be
configures with the correct domain name, and the ghost instance should
still be set to bind to 127.0.0.1:2368, with this value set in the
proxy_pass for nginx.

Currently, changing the `ghost_config_url` also changes the proxy_pass
setting. This commit changes that so that the proxy_pass setting comes
from the `ghost_config_server` values.
@mtpereira mtpereira merged commit 522be8a into mtpereira:master Mar 31, 2016
@mtpereira
Copy link
Owner

Thank you! 👍

@mtpereira
Copy link
Owner

Hey @jesperrasmussen !

Yeah, this is definitely not fresh on my mind. I need to test this locally and get a grasp of it once more.

I'll get back to you as soon as I manage to do it.

Thank you!

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.

None yet

3 participants