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

Remove bad explanaton on HTTPs #4776

Closed
wants to merge 5 commits into from
Closed

Conversation

mastacheata
Copy link
Contributor

The Installation notes for using Nightscout behind a reverse proxy have two conflicting points.
#1 says setting INSECURE_USE_HTTP to false would allow insecure HTTP connections to nightscout, which is plain wrong.
It's also unneccessary to even mention that setting when it's also explained in the second point how and when you should change that setting at all.

sulkaharo and others added 5 commits July 20, 2019 10:06
The Installation notes for using Nightscout behind a reverse proxy have two conflicting points.
#1 says setting INSECURE_USE_HTTP to false would allow insecure HTTP connections to nightscout, which is plain wrong.
It's also unneccessary to even mention that setting when it's also explained in the second point how and when you should change that setting at all.
@mastacheata mastacheata changed the base branch from master to dev July 22, 2019 20:01
@mastacheata mastacheata deleted the patch-1 branch July 22, 2019 20:02
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