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

Fixes HTTPS Error On Logout #140

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

RHOPKINS13
Copy link

Fixes issue #111. Should work for both HTTP and HTTPS.

@technogenus
Copy link

technogenus commented Sep 2, 2017

This error of using HTTP_ROOT instead of HTTPS_HOME occurs throughout this file (and throughout the entire web app, actually). There are 81 replacements needed in this file alone.

Simple find/replace of HTTP_ROOT to HTTPS_HOME basically works.

@RHOPKINS13
Copy link
Author

There's no need to go replacing HTTP_ROOT throughout the whole application. My company has been using Orange Scrum for the past 11 months with HTTPS. We just changed the one reference to HTTP_HOME to HTTPS_HOME and everything works (we might have used https in the url for the HTTP_HOME constant, I can't remember at the moment.) No problems.

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