Uploaded image for project: 'IGB'
  1. IGB
  2. IGBF-2669

Investigate: How does IGB handle Quickload URL redirection from http to https

    Details

    • Type: Task
    • Status: Closed (View Workflow)
    • Priority: Minor
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: None
    • Labels:
      None
    • Story Points:
      1
    • Sprint:
      Fall 7 Dec 14 - Dec 23, Spring 1 2021 Mar 22 - Apr 2, Spring 2 2021 Apr 5 - Apr 16, Spring 6 2022 Mar 14 - Mar 25

      Description

      A lot of Web sites are configured to redirect http to https for most URLs.

      However, it is not clear how IGB handles this when accessing Quickload URLs.

      For example, what happens if IGB tries to contact a site added to IGB as Quickload URL "http://example.com/quickload" and the server is configured to redirect request to "https://example.com/quickload" ?

      Can IGB properly handle the redirect?

      Please investigate.

      Possible strategy:

      • Add an all-new testing Quickload site to BioViz.org (e.g., http://bioviz.org/example_quickload), submit PR for the example and wait for it to merged and deployed
      • Confirm that the redirection is working by opening the "contents.txt" URL in a browser (http://bioviz.org/example_quickload/contents.txt)
      • Deactivate all current Quickload sites in IGB and then re-start IGB
      • Next, add this new Quickload testing URL to IGB using protocol "http" as the quickload address
      • Then try to open the genome from the testing site in IGB
      • Report any errors observed.

      Does IGB appear to be reading and using the testing quickload meta-data files to configure the genome?
      Report on any errors.

        Attachments

        1. example.png
          example.png
          49 kB
        2. http.png
          http.png
          62 kB
        3. https.png
          https.png
          74 kB

          Issue Links

            Activity

            Hide
            omarne Omkar Marne (Inactive) added a comment -

            Dr. [~aloraine] Do you have the reference links or previous tickets of adding new quickload site to bioviz.org?

            Show
            omarne Omkar Marne (Inactive) added a comment - Dr. [~aloraine] Do you have the reference links or previous tickets of adding new quickload site to bioviz.org?
            Hide
            ann.loraine Ann Loraine added a comment -

            Please review or search the commit history of the bioviz.org git repository: https://bitbucket.org/lorainelab/bioviz/src/master/

            Show
            ann.loraine Ann Loraine added a comment - Please review or search the commit history of the bioviz.org git repository: https://bitbucket.org/lorainelab/bioviz/src/master/
            Hide
            omarne Omkar Marne (Inactive) added a comment - - edited

            Hello Dr. Loraine, I have created a sample example_quickload. Please find my PR : https://bitbucket.org/lorainelab/bioviz/pull-requests/86/igbf-2669-example-quicload

            Show
            omarne Omkar Marne (Inactive) added a comment - - edited Hello Dr. Loraine, I have created a sample example_quickload. Please find my PR : https://bitbucket.org/lorainelab/bioviz/pull-requests/86/igbf-2669-example-quicload
            Hide
            ann.loraine Ann Loraine added a comment -

            Change requests:

            1) Change "example_genome" directory name to A_thaliana_Jun_2009
            2) Move "example_quickload" to "htdocs/quickload"
            3) Change name of "example_quickload" to "example"
            4) Edit "annots.xml" to include only one file tag
            5) Squash commits on the branch so that only the smaller version of annots.xml file does will get merged in

            Show
            ann.loraine Ann Loraine added a comment - Change requests: 1) Change "example_genome" directory name to A_thaliana_Jun_2009 2) Move "example_quickload" to "htdocs/quickload" 3) Change name of "example_quickload" to "example" 4) Edit "annots.xml" to include only one file tag 5) Squash commits on the branch so that only the smaller version of annots.xml file does will get merged in
            Hide
            omarne Omkar Marne (Inactive) added a comment -

            Hello Dr. [~aloraine] Please see the requested changes below.

            https://bitbucket.org/lorainelab/bioviz/pull-requests/86/igbf-2669

            Show
            omarne Omkar Marne (Inactive) added a comment - Hello Dr. [~aloraine] Please see the requested changes below. https://bitbucket.org/lorainelab/bioviz/pull-requests/86/igbf-2669
            Hide
            ann.loraine Ann Loraine added a comment -

            Please get rid of initial commit that added enormous xml file to repo.

            Show
            ann.loraine Ann Loraine added a comment - Please get rid of initial commit that added enormous xml file to repo.
            Hide
            omarne Omkar Marne (Inactive) added a comment -
            Show
            omarne Omkar Marne (Inactive) added a comment - Hello Dr. Loraine Please find my new pull request : https://bitbucket.org/lorainelab/bioviz/pull-requests/87/igbf-2669-new-sample-quickload
            Hide
            ann.loraine Ann Loraine added a comment -

            Thanks - looks good. Proceeding to merge.

            Show
            ann.loraine Ann Loraine added a comment - Thanks - looks good. Proceeding to merge.
            Hide
            ann.loraine Ann Loraine added a comment -

            New example quickload is deployed to https://www.bioviz.org/quickload/example/

            Moving back to "to-do" for Omkar Marne to use the new quickload to test as noted in the Description.

            Show
            ann.loraine Ann Loraine added a comment - New example quickload is deployed to https://www.bioviz.org/quickload/example/ Moving back to "to-do" for Omkar Marne to use the new quickload to test as noted in the Description.
            Hide
            ann.loraine Ann Loraine added a comment -

            New example quickload is deployed to https://www.bioviz.org/quickload/example/

            Moving back to "to-do" for Omkar Marne to use the new quickload to test as noted in the Description.

            Show
            ann.loraine Ann Loraine added a comment - New example quickload is deployed to https://www.bioviz.org/quickload/example/ Moving back to "to-do" for Omkar Marne to use the new quickload to test as noted in the Description.
            Hide
            omarne Omkar Marne (Inactive) added a comment -

            Test results after adding above quickload as data source:

            IGB won't allow http protocol as data source as the server is set to automatically redirect from http to https. If the user uses the http quickload url then IGB won't follow the redirect and the quickload will not be added.

            Show
            omarne Omkar Marne (Inactive) added a comment - Test results after adding above quickload as data source: IGB won't allow http protocol as data source as the server is set to automatically redirect from http to https. If the user uses the http quickload url then IGB won't follow the redirect and the quickload will not be added.
            Hide
            nfreese Nowlan Freese added a comment - - edited

            Testing on a quickload hosted on bitbucket (which redirects from http to https as tested using Google Chrome and the contents.txt file) also fails to work in IGB if the quickload URL is set to http. Along with Omkar's testing it would appear that IGB does not follow the redirect from http to https for Quickloads.

            Console logs:
            10:18:29.900 INFO o.l.i.q.QuickloadDataProvider - Initializing Quickload Server http://bitbucket.org/nfreese/quickload-genome-in-a-bottle/raw/release-v1.0.0/
            10:18:30.036 WARN o.l.i.quickload.util.QuickloadUtils - Could not read contents.txt from: http://bitbucket.org/nfreese/quickload-genome-in-a-bottle/raw/release-v1.0.0/contents.txt
            10:18:30.037 WARN o.l.i.q.QuickloadDataProvider - Missing required quickload file, or could not reach source. This quickloak source will be disabled for this session.
            10:18:30.069 WARN o.l.i.quickload.util.QuickloadUtils - Optional quickload synonyms.txt file could not be loaded from http://bitbucket.org/nfreese/quickload-genome-in-a-bottle/raw/release-v1.0.0/synonyms.txt
            10:18:30.101 WARN o.l.i.quickload.util.QuickloadUtils - Optional species.txt could not be loaded from: http://bitbucket.org/nfreese/quickload-genome-in-a-bottle/raw/release-v1.0.0/species.txt

            Show
            nfreese Nowlan Freese added a comment - - edited Testing on a quickload hosted on bitbucket (which redirects from http to https as tested using Google Chrome and the contents.txt file) also fails to work in IGB if the quickload URL is set to http. Along with Omkar's testing it would appear that IGB does not follow the redirect from http to https for Quickloads. Console logs: 10:18:29.900 INFO o.l.i.q.QuickloadDataProvider - Initializing Quickload Server http://bitbucket.org/nfreese/quickload-genome-in-a-bottle/raw/release-v1.0.0/ 10:18:30.036 WARN o.l.i.quickload.util.QuickloadUtils - Could not read contents.txt from: http://bitbucket.org/nfreese/quickload-genome-in-a-bottle/raw/release-v1.0.0/contents.txt 10:18:30.037 WARN o.l.i.q.QuickloadDataProvider - Missing required quickload file, or could not reach source. This quickloak source will be disabled for this session. 10:18:30.069 WARN o.l.i.quickload.util.QuickloadUtils - Optional quickload synonyms.txt file could not be loaded from http://bitbucket.org/nfreese/quickload-genome-in-a-bottle/raw/release-v1.0.0/synonyms.txt 10:18:30.101 WARN o.l.i.quickload.util.QuickloadUtils - Optional species.txt could not be loaded from: http://bitbucket.org/nfreese/quickload-genome-in-a-bottle/raw/release-v1.0.0/species.txt
            Hide
            nfreese Nowlan Freese added a comment -

            I have created a new ticket (IGBF-3103) to address the redirect issue.

            Closing ticket.

            Show
            nfreese Nowlan Freese added a comment - I have created a new ticket ( IGBF-3103 ) to address the redirect issue. Closing ticket.

              People

              • Assignee:
                omarne Omkar Marne (Inactive)
                Reporter:
                ann.loraine Ann Loraine
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: