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

Reverse Proxy Requires Docker Environmental Variable #12

Closed
elmigbot opened this issue Apr 1, 2020 · 5 comments
Closed

Reverse Proxy Requires Docker Environmental Variable #12

elmigbot opened this issue Apr 1, 2020 · 5 comments

Comments

@elmigbot
Copy link

elmigbot commented Apr 1, 2020

Problem/Motivation

BookStack needs to know that HTTPS is being used behind reverse proxy so that mixed http/https calls are not made. There is a docker parameter that can be passed in to allow this to happen.

-e APP_URL=http://your.site.here.xyz

Without this, bookStack fails to load properly as noted here:

Proposed changes

Can we add this parameter to the addon config options?

@addons-assistant
Copy link

👋 Thanks for opening your first issue here! If you're reporting a 🐛 bug, please make sure you include steps to reproduce it. Also, logs, error messages and information about your hardware might be useful.

@sinclairpaul
Copy link
Member

Just to confirm the setup, you have an external reverse proxy serving on SSL, while the addon is running on HTTP?

@sinclairpaul
Copy link
Member

I will add, you may need to clear any cookies for this, and that any attachments uploaded will fail to display when the url is changed due to the way the link is stored.

@elmigbot
Copy link
Author

elmigbot commented Apr 2, 2020

Works great! Thank you very much.

@addons-assistant
Copy link

This thread has been automatically locked because it has not had recent activity. Please open a new issue for related bugs and link to relevant comments in this thread.

@addons-assistant addons-assistant bot locked as resolved and limited conversation to collaborators May 2, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants