Details
-
Type:
Bug
-
Status: Closed (View Workflow)
-
Priority:
Major
-
Resolution: Done
-
Affects Version/s: None
-
Fix Version/s: None
-
Labels:None
-
Story Points:2
-
Epic Link:
-
Sprint:Fall 2: 28 Sep - 9 Oct, Fall 3: Oct 12 - Oct 23
Description
Situation: BioViz Connect settings.ini has a variable IS_DEV_ENVIRONMENT that can be set to true/false for development/production. If set to true and a valid CyVerse username and password are also included in settings.ini, then loadlogin method within views.py is supposed to use the username and password to obtain an access token without using the normal CAS login. However, instead of using the value of IS_DEV_ENVIRONMENT loadlogin is hardcoded with False. This is due to a bug where if it is set to True the access token appears to be obtained, but then the user is redirected to the connect.bioviz CAS login page instead of their dev server. This may have to do with some logic changes we made to logging out the user and redirecting them to connect.bioviz.org whenever an issue is encountered - such as an expired access token.
Task: Investigate and fix the issue so that loadlogin uses the IS_DEV_ENVIRONMENT variable properly.
Attachments
Issue Links
- relates to
-
IGBF-2521 Add dev setting to BioViz Connect playbook
-
- Closed
-
Activity
| Field | Original Value | New Value |
|---|---|---|
| Epic Link | IGBF-2376 [ 18533 ] |
| Status | To-Do [ 10305 ] | In Progress [ 3 ] |
| Status | In Progress [ 3 ] | Needs 1st Level Review [ 10005 ] |
| Status | Needs 1st Level Review [ 10005 ] | First Level Review in Progress [ 10301 ] |
| Status | First Level Review in Progress [ 10301 ] | Needs 1st Level Review [ 10005 ] |
| Status | Needs 1st Level Review [ 10005 ] | First Level Review in Progress [ 10301 ] |
| Status | First Level Review in Progress [ 10301 ] | Needs 1st Level Review [ 10005 ] |
| Status | Needs 1st Level Review [ 10005 ] | First Level Review in Progress [ 10301 ] |
| Status | First Level Review in Progress [ 10301 ] | Ready for Pull Request [ 10304 ] |
| Status | Ready for Pull Request [ 10304 ] | Pull Request Submitted [ 10101 ] |
| Status | Pull Request Submitted [ 10101 ] | Reviewing Pull Request [ 10303 ] |
| Status | Reviewing Pull Request [ 10303 ] | Merged Needs Testing [ 10002 ] |
| Assignee | Chaitanya Kintali [ chaitanya ] | Nowlan Freese [ nfreese ] |
| Status | Merged Needs Testing [ 10002 ] | Post-merge Testing In Progress [ 10003 ] |
| Sprint | Fall 2: 28 Sep - 9 Oct [ 104 ] | Fall 2: 28 Sep - 9 Oct, Fall 3: Oct 12 - Oct 23 [ 104, 106 ] |
| Rank | Ranked higher |
| Status | Post-merge Testing In Progress [ 10003 ] | Merged Needs Testing [ 10002 ] |
| Status | Merged Needs Testing [ 10002 ] | Post-merge Testing In Progress [ 10003 ] |
Tested using personal AWS account using BioViz Connect ansible playbook. When IS_DEV_ENVIRONMENT in the settings.ini is set to True the user is automatically logged in using the USERNAME and PASSWORD variables in the settings.ini. When IS_DEV_ENVIRONMENT is set to False the normal CyVerse oauth login occurs. I tried this on an ec2 with an elastic ip set to nowlantest.bioviz.org and on an ec2 not associated with a domain name. Both worked when IS_DEV_ENVIRONMENT was set to True. This is useful for future developers as the normal CyVerse oauth login requires a domain name that ends with bioviz.org.
Closing issue.
| Assignee | Nowlan Freese [ nfreese ] | Chaitanya Kintali [ chaitanya ] |
| Resolution | Done [ 10000 ] | |
| Status | Post-merge Testing In Progress [ 10003 ] | Closed [ 6 ] |
Pull Request Link:
https://bitbucket.org/nfreese/django-cyversedev/pull-requests/195/igbf-2549/diff
Moving the ticket to Needs testing.