-
-
Notifications
You must be signed in to change notification settings - Fork 15
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
Added LAN Guidance in readme-vars.yml #26
base: master
Are you sure you want to change the base?
Conversation
Myself and some other users reported issues with the software working out of the box on LAN only. This is resolved by commenting out the proxy_set_header in the nginx config. Adding some clear guidance on this would reduce issues opened and help future users
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.
Thanks for opening this pull request! Be sure to follow the pull request template!
I am a bot, here are the test results for this PR:
|
I am a bot, here are the test results for this PR:
|
I am a bot, here are the test results for this PR:
|
I am a bot, here are the test results for this PR:
|
I am a bot, here are the test results for this PR:
|
I am a bot, here are the test results for this PR:
|
Resolves #7 for good.
Description:
Myself and some other users reported issues with the software working out of the box on LAN only. This is resolved by commenting out the proxy_set_header in the nginx config. Adding some clear guidance on this would reduce issues opened and help future users
Benefits of this PR and context:
Increase satisfaction of userbase as more likely to work on simpler setups, eg, local networks without proxy
How Has This Been Tested?
Only Doc Change
Source / References: