Details
-
Type:
Task
-
Status: Closed (View Workflow)
-
Priority:
Major
-
Resolution: Done
-
Affects Version/s: None
-
Fix Version/s: None
-
Labels:
-
Story Points:0.5
-
Epic Link:
-
Sprint:Summer 5: 3 Aug - 14 Aug, Summer 6: 17 Aug - 28 Aug
Description
Chirag looked at a week of BioViz.org Web logs and found that around 9% of visits to our "welcome galaxy user" IGB endpoint page are using the Safari browser.
For this task:
Please check the current release of the Galaxy code base. Find out: does the external viewers XML hit an "https" endpoint on Bioviz or does it hit an "http" endpoint.
If it is currently hitting an http endpoint on our site - yay! That's what we want, for now. If this is the case, please then modify the bioviz playbooks to STOP redirecting all http web traffic to https.
However, if you can think of a good and bullet-proof way to turn off redirection for JUST galaxy.html, do that instead. However, be aware that this type of thing is very hard to test, and it is also hard to avoid affecting other pages, due to the use of regular expressions in the redirection directive.
Attachments
Issue Links
- relates to
-
IGBF-2987 Fix Galaxy IGB CORS issue
-
- Closed
-
-
-
- Closed
-
-
-
- Closed
-
-
-
- Closed
-
-
-
- Closed
-
-
-
- Closed
-
-
-
- Closed
-
-
-
- Closed
-
Activity
| Field | Original Value | New Value |
|---|---|---|
| Assignee | Ann Loraine [ aloraine ] |
| Sprint | Summer 5: 3 Aug - 14 Aug [ 100 ] |
| Epic Link | IGBF-1765 [ 17855 ] |
| Description |
Chirag looked at a week of BioViz.org Web logs and found that around 9% of visits to our "welcome galaxy user" IGB endpoint page are using the Safari browser.
For this task: Please check the current release of the Galaxy code base. Find out: does the external viewers XML hit an "https" endpoint on Bioviz or does it hit an "http" endpoint. If it is hitting and http endpoint - yay! That's what we want, for now. If yes, then modify the bioviz playbooks to STOP redirecting all http web traffic to https. If it is possible to turn off redirection for JUST galaxy.html, then do that instead. However, it is very hard to test and hard to avoid affecting other pages, due to the need to use regular expressions. |
Chirag looked at a week of BioViz.org Web logs and found that around 9% of visits to our "welcome galaxy user" IGB endpoint page are using the Safari browser.
For this task: Please check the current release of the Galaxy code base. Find out: does the external viewers XML hit an "https" endpoint on Bioviz or does it hit an "http" endpoint. If it is currently hitting an http endpoint on our site - yay! That's what we want, for now. If this is the case, please then modify the bioviz playbooks to STOP redirecting all http web traffic to https. However, if you can think of a good and bullet-proof way to turn off redirection for JUST galaxy.html, do that instead. However, be aware that this type of thing is very hard to test, and it is also hard to avoid affecting other pages, due to the use of regular expressions in the redirection directive. |
| Summary | Turn off redirection to https in bioviz-playbooks | Turn off redirection to https for galaxy bridge in bioviz-playbooks |
| Story Points | 0.4 | 0.5 |
| Labels | Advanced |
| Rank | Ranked higher |
| Assignee | Philip Badzuh [ pbadzuh ] |
| Status | To-Do [ 10305 ] | In Progress [ 3 ] |
| Status | In Progress [ 3 ] | Needs 1st Level Review [ 10005 ] |
| Assignee | Philip Badzuh [ pbadzuh ] |
Thank you. Please submit PR.
| Status | Needs 1st Level Review [ 10005 ] | First Level Review in Progress [ 10301 ] |
| Status | First Level Review in Progress [ 10301 ] | Ready for Pull Request [ 10304 ] |
| Assignee | Philip Badzuh [ pbadzuh ] |
Sure, no problem. Please see the PR below:
https://bitbucket.org/lorainelab/bioviz-playbooks/pull-requests/5/igbf-2505-ensure-bioviz-galaxy-connections/diff
| Status | Ready for Pull Request [ 10304 ] | Pull Request Submitted [ 10101 ] |
| Assignee | Philip Badzuh [ pbadzuh ] |
Merged and ready for testing.
| Status | Pull Request Submitted [ 10101 ] | Reviewing Pull Request [ 10303 ] |
| Status | Reviewing Pull Request [ 10303 ] | Merged Needs Testing [ 10002 ] |
| Sprint | Summer 5: 3 Aug - 14 Aug [ 100 ] | Summer 5: 3 Aug - 14 Aug, Summer 6: 17 Aug - 28 Aug [ 100, 101 ] |
| Rank | Ranked higher |
Changes are now deployed on BioViz.org.
To test: Check that Galaxy bridge is functional in Safari and Chrome.
For instructions, see IGB release testing protocols - https://wiki.transvar.org/display/ITD/Galaxy
| Assignee | Shamika Gajanan Kulkarni [ shamika ] |
| Status | Merged Needs Testing [ 10002 ] | Post-merge Testing In Progress [ 10003 ] |
Tested this ticket.
Followed the instructions on https://wiki.transvar.org/display/ITD/Galaxy.
The Galaxy bridge is working fine in Chrome. It loads over http.
Someone needs to test this on a Mac.
cc: Ann Loraine
| Status | Post-merge Testing In Progress [ 10003 ] | Merged Needs Testing [ 10002 ] |
| Assignee | Shamika Gajanan Kulkarni [ shamika ] |
Could you recruit someone to help with this and walk through it real-time? Anyone with a Mac – both Noor and Chester have Macs, I believe.
cc: Shamika Gajanan Kulkarni
| Assignee | Chester Dias [ cdias1 ] |
| Status | Merged Needs Testing [ 10002 ] | Post-merge Testing In Progress [ 10003 ] |
Tested on Mac
Chrome
Working as per instructions mentioned on https://wiki.transvar.org/display/ITD/Galaxy
Safari
Working as per instructions mentioned on https://wiki.transvar.org/display/ITD/Galaxy
| Resolution | Done [ 10000 ] | |
| Status | Post-merge Testing In Progress [ 10003 ] | Closed [ 6 ] |
| Assignee | Chester Dias [ cdias1 ] | Philip Badzuh [ pbadzuh ] |
The galaxy page does indeed currently communicate with the IGB API over http. Please see the changes I have made to the bioviz apache playbook to ensure the galaxy page always loads over http.
https://bitbucket.org/pbadzuh/bioviz-playbooks_pbdev/branch/IGBF-2505#diff