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

Identify hard-coded URLs in IGB code base

    Details

    • Story Points:
      1
    • Sprint:
      B - Summer 2018, Summer 2018 Part 2, Summer 2018 Part 3, Fall 2018 1, Fall 2018 Sprint 2

      Description

      IGB appears hard-coded to request resources from BioViz.org and a few other sites.

      This makes testing and developing IGB challenging for developers because they need access to the access_logs of BioViz to add new features or fix bugs. It also makes updating URLs difficult and may slow down startup if IGB tries to access resources that no longer exist.

      Task is:

      Identify all places in IGB code base with URLs. Document these:

      • Report name of file with hard-coded URL (.java or other resources)
      • Report URL

      Request BioViz.org access_log from Dr. Loraine. Do not attach here to respect users' privacy.

        Attachments

          Activity

          ann.loraine Ann Loraine created issue -
          ann.loraine Ann Loraine made changes -
          Field Original Value New Value
          Description IGB appears hard-coded to request resources from BioViz.org.

          This makes testing and developing IGB challenging for developers because they need access to the access_logs of BioViz to add new features or fix bugs.

          * Identify all places in IGB code bas with hard-coded URLs. Document here.

          Ideas for how to fix/improve:

          * IGB installer Install4J uses codeBaseUrl setting in its configuration file to configure future updates. Use this to determine IGB code base URL.

          * Create a new IGB Service (new module) that provides URLs on demand for key IGB functions, e.g., IGB Help Page, IGB Code Base, etc.
          IGB appears hard-coded to request resources from BioViz.org and others.

          This makes testing and developing IGB challenging for developers because they need access to the access_logs of BioViz to add new features or fix bugs. It also makes updating URLs difficult and may slow down startup if IGB tries to access resources that no longer exist.

          * Identify all places in IGB code base with hard-coded URLs. Document these here.
          * Develop a plan for how to improve the situation.

          Thought for how to fix/improve:

          * IGB installer Install4J uses codeBaseUrl setting in its configuration file to configure future updates. Use this to determine IGB BioViz code base URL.

          * Create a new IGB Service (new module) that provides URLs on demand for key IGB functions, e.g., IGB Help Page, IGB Code Base, etc.
          ann.loraine Ann Loraine made changes -
          Description IGB appears hard-coded to request resources from BioViz.org and others.

          This makes testing and developing IGB challenging for developers because they need access to the access_logs of BioViz to add new features or fix bugs. It also makes updating URLs difficult and may slow down startup if IGB tries to access resources that no longer exist.

          * Identify all places in IGB code base with hard-coded URLs. Document these here.
          * Develop a plan for how to improve the situation.

          Thought for how to fix/improve:

          * IGB installer Install4J uses codeBaseUrl setting in its configuration file to configure future updates. Use this to determine IGB BioViz code base URL.

          * Create a new IGB Service (new module) that provides URLs on demand for key IGB functions, e.g., IGB Help Page, IGB Code Base, etc.
          IGB appears hard-coded to request resources from BioViz.org and others.

          This makes testing and developing IGB challenging for developers because they need access to the access_logs of BioViz to add new features or fix bugs. It also makes updating URLs difficult and may slow down startup if IGB tries to access resources that no longer exist.

          * Identify all places in IGB code base with hard-coded URLs. Document these here.
          * Develop & implement solution.

          Thought for how to fix/improve:

          * IGB installer Install4J uses codeBaseUrl setting in its configuration file to configure future updates. Use this to determine IGB BioViz code base URL.

          * Create a new IGB Service (new module) that provides URLs on demand for key IGB functions, e.g., IGB Help Page, IGB Code Base, etc.
          ann.loraine Ann Loraine made changes -
          Description IGB appears hard-coded to request resources from BioViz.org and others.

          This makes testing and developing IGB challenging for developers because they need access to the access_logs of BioViz to add new features or fix bugs. It also makes updating URLs difficult and may slow down startup if IGB tries to access resources that no longer exist.

          * Identify all places in IGB code base with hard-coded URLs. Document these here.
          * Develop & implement solution.

          Thought for how to fix/improve:

          * IGB installer Install4J uses codeBaseUrl setting in its configuration file to configure future updates. Use this to determine IGB BioViz code base URL.

          * Create a new IGB Service (new module) that provides URLs on demand for key IGB functions, e.g., IGB Help Page, IGB Code Base, etc.
          IGB appears hard-coded to request resources from BioViz.org and others.

          This makes testing and developing IGB challenging for developers because they need access to the access_logs of BioViz to add new features or fix bugs. It also makes updating URLs difficult and may slow down startup if IGB tries to access resources that no longer exist.

          * Identify all places in IGB code base with hard-coded URLs. Document these here.

          * Develop & implement solution.

          Thought for how to fix/improve:

          * IGB installer Install4J uses codeBaseUrl setting in its configuration file to configure future updates. Use this to determine IGB BioViz code base URL.

          * Create a new IGB Service (new module) that provides URLs on demand for key IGB functions, e.g., IGB Help Page, IGB Code Base, etc.
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          ann.loraine Ann Loraine made changes -
          Sprint B - Summer 2018 [ 48 ] B - Summer 2018, Summer 2018 Part 2 [ 48, 49 ]
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          ann.loraine Ann Loraine made changes -
          Summary identify and parameterize all references to BioViz.org in IGB code base identify hard-coded URLs in IGB code base
          ann.loraine Ann Loraine made changes -
          Description IGB appears hard-coded to request resources from BioViz.org and others.

          This makes testing and developing IGB challenging for developers because they need access to the access_logs of BioViz to add new features or fix bugs. It also makes updating URLs difficult and may slow down startup if IGB tries to access resources that no longer exist.

          * Identify all places in IGB code base with hard-coded URLs. Document these here.

          * Develop & implement solution.

          Thought for how to fix/improve:

          * IGB installer Install4J uses codeBaseUrl setting in its configuration file to configure future updates. Use this to determine IGB BioViz code base URL.

          * Create a new IGB Service (new module) that provides URLs on demand for key IGB functions, e.g., IGB Help Page, IGB Code Base, etc.
          IGB appears hard-coded to request resources from BioViz.org and a few other sites.

          This makes testing and developing IGB challenging for developers because they need access to the access_logs of BioViz to add new features or fix bugs. It also makes updating URLs difficult and may slow down startup if IGB tries to access resources that no longer exist.

          Task is:

          Identify all places in IGB code base with URLs. Document these:

          * Report name of file with hard-coded URL (.java or other resources)
          * Report URL

          Request BioViz.org access_log from Dr. Loraine. Do not attach here to respect users' privacy.
          ann.loraine Ann Loraine made changes -
          Assignee Ann Loraine [ aloraine ] Sneha Ramesh Watharkar [ jdaly ]
          ann.loraine Ann Loraine made changes -
          Assignee Sneha Ramesh Watharkar [ jdaly ]
          ann.loraine Ann Loraine made changes -
          Assignee Sneha Ramesh Watharkar [ jdaly ]
          Hide
          ann.loraine Ann Loraine added a comment -

          Tip:

          • Configure your laptop to send all requests for bioviz.org or wiki.transvar.org Web pages to your testing site
          • On Unix, you would edit /etc/hosts

          IGB already is designed to request specific files from Quickload hosts (these are "baked in" and less likely to be optimizable). For example, IGB will try to retrieve a file called synonyms.txt from a Quickload site. This file is optional. If it not there, it's not considered an error.

          Show
          ann.loraine Ann Loraine added a comment - Tip: Configure your laptop to send all requests for bioviz.org or wiki.transvar.org Web pages to your testing site On Unix, you would edit /etc/hosts IGB already is designed to request specific files from Quickload hosts (these are "baked in" and less likely to be optimizable). For example, IGB will try to retrieve a file called synonyms.txt from a Quickload site. This file is optional. If it not there, it's not considered an error.
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          ann.loraine Ann Loraine made changes -
          Summary identify hard-coded URLs in IGB code base Identify hard-coded URLs in IGB code base
          ann.loraine Ann Loraine made changes -
          Assignee Sneha Ramesh Watharkar [ jdaly ] Kiran Korey [ kkorey ]
          kkorey Kiran Korey (Inactive) made changes -
          Status Open [ 1 ] In Progress [ 3 ]
          kkorey Kiran Korey (Inactive) made changes -
          Attachment hard_coded_urls_formatted.txt [ 14123 ]
          Attachment hard_coded_urls_original.txt [ 14124 ]
          Hide
          kkorey Kiran Korey (Inactive) added a comment -

          Please find the attached hard_coded_urls_formatted.txt file which contains references to the hardcoded URLs which are relevant and hard_coded_urls_original.txt which contains all references to the string "http://".

          Show
          kkorey Kiran Korey (Inactive) added a comment - Please find the attached hard_coded_urls_formatted.txt file which contains references to the hardcoded URLs which are relevant and hard_coded_urls_original.txt which contains all references to the string "http://".
          kkorey Kiran Korey (Inactive) made changes -
          Attachment hard_coded_https_urls.txt [ 14125 ]
          kkorey Kiran Korey (Inactive) made changes -
          Assignee Kiran Korey [ kkorey ] Sneha Ramesh Watharkar [ jdaly ]
          sneha Sneha Ramesh Watharkar (Inactive) made changes -
          Status In Progress [ 3 ] Open [ 1 ]
          sneha Sneha Ramesh Watharkar (Inactive) made changes -
          Status Open [ 1 ] In Progress [ 3 ]
          Hide
          sneha Sneha Ramesh Watharkar (Inactive) added a comment -

          Followed the recommendations provided by Professor [~aloraine] and made changes to the files which contains hard coded URLs.
          Moving it to 1st Level review column.

          Show
          sneha Sneha Ramesh Watharkar (Inactive) added a comment - Followed the recommendations provided by Professor [~aloraine] and made changes to the files which contains hard coded URLs. Moving it to 1st Level review column.
          sneha Sneha Ramesh Watharkar (Inactive) made changes -
          Assignee Sneha Ramesh Watharkar [ jdaly ]
          sneha Sneha Ramesh Watharkar (Inactive) made changes -
          Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
          Hide
          ann.loraine Ann Loraine added a comment -

          Add link to branch.

          Show
          ann.loraine Ann Loraine added a comment - Add link to branch.
          ann.loraine Ann Loraine made changes -
          Assignee Sneha Ramesh Watharkar [ jdaly ]
          Show
          sneha Sneha Ramesh Watharkar (Inactive) added a comment - You can find the work in this branch : https://bitbucket.org/swathark/integrated-genome-browser/branch/IGBF-1348#diff
          sneha Sneha Ramesh Watharkar (Inactive) made changes -
          Assignee Sneha Ramesh Watharkar [ jdaly ] Ann Loraine [ aloraine ]
          sneha Sneha Ramesh Watharkar (Inactive) made changes -
          Rank Ranked higher
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          ann.loraine Ann Loraine made changes -
          Sprint B - Summer 2018, Summer 2018 Part 2 [ 48, 49 ] B - Summer 2018, Summer 2018 Part 2, Summer 2018 Part 3 [ 48, 49, 50 ]
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          ann.loraine Ann Loraine made changes -
          Status Needs 1st Level Review [ 10005 ] Reviewing [ 10301 ]
          Hide
          ann.loraine Ann Loraine added a comment -

          Following change is needed before pull request can be merged:

          Show
          ann.loraine Ann Loraine added a comment - Following change is needed before pull request can be merged: Change: "updates= http://bioviz.org/igb/releases/igb-9.0.1/updates.xml " to "updates= http://bioviz.org/igb/releases/current/updates.xml " in plugins/Update/src/main/resources/updates.properties
          ann.loraine Ann Loraine made changes -
          Status Reviewing [ 10301 ] Open [ 1 ]
          ann.loraine Ann Loraine made changes -
          Assignee Ann Loraine [ aloraine ] Sneha Ramesh Watharkar [ jdaly ]
          Hide
          sneha Sneha Ramesh Watharkar (Inactive) added a comment -

          Done with the above fix.

          Show
          sneha Sneha Ramesh Watharkar (Inactive) added a comment - Done with the above fix.
          sneha Sneha Ramesh Watharkar (Inactive) made changes -
          Assignee Sneha Ramesh Watharkar [ jdaly ] Ann Loraine [ aloraine ]
          sneha Sneha Ramesh Watharkar (Inactive) made changes -
          Status Open [ 1 ] Needs 1st Level Review [ 10005 ]
          ann.loraine Ann Loraine made changes -
          Status Needs 1st Level Review [ 10005 ] Ready for Pull Request [ 10304 ]
          Hide
          ann.loraine Ann Loraine added a comment -

          Ready for pull request.

          Show
          ann.loraine Ann Loraine added a comment - Ready for pull request.
          ann.loraine Ann Loraine made changes -
          Assignee Ann Loraine [ aloraine ] Sneha Ramesh Watharkar [ jdaly ]
          sneha Sneha Ramesh Watharkar (Inactive) made changes -
          Status Ready for Pull Request [ 10304 ] Pull Request Submitted [ 10101 ]
          Hide
          sneha Sneha Ramesh Watharkar (Inactive) added a comment -

          PR submitted.

          Show
          sneha Sneha Ramesh Watharkar (Inactive) added a comment - PR submitted.
          sneha Sneha Ramesh Watharkar (Inactive) made changes -
          Assignee Sneha Ramesh Watharkar [ jdaly ] Ann Loraine [ aloraine ]
          ann.loraine Ann Loraine made changes -
          Status Pull Request Submitted [ 10101 ] Needs Testing [ 10002 ]
          ann.loraine Ann Loraine made changes -
          Assignee Ann Loraine [ aloraine ]
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          ann.loraine Ann Loraine made changes -
          Sprint B - Summer 2018, Summer 2018 Part 2, Summer 2018 Part 3 [ 48, 49, 50 ] B - Summer 2018, Summer 2018 Part 2, Summer 2018 Part 3, Fall 2018 1 [ 48, 49, 50, 51 ]
          ann.loraine Ann Loraine made changes -
          Rank Ranked lower
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          ann.loraine Ann Loraine made changes -
          Sprint B - Summer 2018, Summer 2018 Part 2, Summer 2018 Part 3, Fall 2018 1 [ 48, 49, 50, 51 ] B - Summer 2018, Summer 2018 Part 2, Summer 2018 Part 3, Fall 2018 1, Fall 2018 Sprint 2 [ 48, 49, 50, 51, 52 ]
          ann.loraine Ann Loraine made changes -
          Rank Ranked lower
          Hide
          ieclabau Ivory Blakley (Inactive) added a comment - - edited

          I don't know how to test this issue.
          In the changes that were made for this issue, we removed things that we were sure were not used.
          So I am moving it to done.
          Reassigning to Sneha because she did the work.

          Show
          ieclabau Ivory Blakley (Inactive) added a comment - - edited I don't know how to test this issue. In the changes that were made for this issue, we removed things that we were sure were not used. So I am moving it to done. Reassigning to Sneha because she did the work.
          ieclabau Ivory Blakley (Inactive) made changes -
          Resolution Done [ 10000 ]
          Status Needs Testing [ 10002 ] Closed [ 6 ]
          ieclabau Ivory Blakley (Inactive) made changes -
          Assignee Sneha Ramesh Watharkar [ jdaly ]
          ann.loraine Ann Loraine made changes -
          Workflow Loraine Lab Workflow [ 18040 ] Fall 2019 Workflow Update [ 19915 ]
          ann.loraine Ann Loraine made changes -
          Workflow Fall 2019 Workflow Update [ 19915 ] Revised Fall 2019 Workflow Update [ 22034 ]

            People

            • Assignee:
              sneha Sneha Ramesh Watharkar (Inactive)
              Reporter:
              ann.loraine Ann Loraine
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: