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

Rework IGB Trust Certificate Dialog

        Details

        • Type: Improvement
        • Status: Closed (View Workflow)
        • Priority: Major
        • Resolution: Done
        • Affects Version/s: None
        • Fix Version/s: None
        • Labels:
          None

          Description

          Situation: If a user attempts to load a file in IGB via URL through HTTPS and there is no issue with the certificate, IGB will load the file. If there is an issue with the certificate, IGB will ask the user if they trust the certificate. If the user confirms that they trust the certificate, the certificate is still considered invalid and the connection refused.

          Possible Resolution: Either IGB needs to honor the user's decision or not ask the user to validate the certificate and inform them that there is an issue with the certificate.

          Task: Understand the architecture and event flow that triggers it. Implement a fix that will lead to either outcome above.

          This issue is closely related to IGBF-1362.

          Important files:
          IGBTrustManager.java
          LocalUrlCacher.java
          GeneralLoadUtils.java

            Attachments

              Issue Links

              relates to

              Improvement - An improvement or enhancement to an existing feature or task. IGBF-1362 Investigate and improve when IGB shows certificate dialog

              • Major - Major loss of function.
              • Closed

              Connection Issue - IGBF-2008 IGB fails to connect to public CyVerse Data

              • Blocker - Blocks development and/or testing work, production could not run.
              • Closed

                Activity

                Ascending order - Click to sort in descending order
                nfreese Nowlan Freese created issue -
                nfreese Nowlan Freese made changes -
                Field Original Value New Value
                Link This issue relates to IGBF-1362 [ IGBF-1362 ]
                nfreese Nowlan Freese made changes -
                Link This issue relates to IGBF-2008 [ IGBF-2008 ]
                nfreese Nowlan Freese made changes -
                Description Situation: If a user attempts to load a file in IGB via URL through HTTPS and there is no issue with the certificate, IGB will load the file. If there is an issue with the certificate, IGB will ask the user if they trust the certificate. If the user confirms that they trust the certificate, the certificate is still considered invalid and the connection refused.

                *Either IGB needs to honor the user's decision or not ask the user to validate the certificate and inform them that there is an issue with the certificate.

                Task: Understand the architecture and event flow that triggers it. Implement a fix that will lead to either outcome above.

                This issue is closely related to IGBF-1362.

                Important files:
                IGBTrustManager.java
                LocalUrlCacher.java
                GeneralLoadUtils.java                 		Situation: If a user attempts to load a file in IGB via URL through HTTPS and there is no issue with the certificate, IGB will load the file. If there is an issue with the certificate, IGB will ask the user if they trust the certificate. If the user confirms that they trust the certificate, the certificate is still considered invalid and the connection refused.

                Possible Resolution: Either IGB needs to honor the user's decision or not ask the user to validate the certificate and inform them that there is an issue with the certificate.

                Task: Understand the architecture and event flow that triggers it. Implement a fix that will lead to either outcome above.

                This issue is closely related to IGBF-1362.

                Important files:
                IGBTrustManager.java
                LocalUrlCacher.java
                GeneralLoadUtils.java
                noor91zahara Noor Zahara (Inactive) made changes -
                Assignee Noor Zahara [ noor91zahara ]
                ann.loraine Ann Loraine made changes -
                Assignee Noor Zahara [ noor91zahara ]
                Hide
                Permalink
                ann.loraine Ann Loraine added a comment -

                Moving to next sprint.

                Show
                ann.loraine Ann Loraine added a comment - Moving to next sprint.
                ann.loraine Ann Loraine made changes -
                Sprint Fall 2019 Sprint 2 [ 73 ] Fall 2019 Sprint 4 [ 74 ]
                ann.loraine Ann Loraine made changes -
                Rank Ranked higher
                ann.loraine Ann Loraine made changes -
                Rank Ranked higher
                ann.loraine Ann Loraine made changes -
                Sprint Fall 2019 Sprint 3 [ 74 ] Fall 2019 Sprint 4 [ 75 ]
                ann.loraine Ann Loraine made changes -
                Rank Ranked lower
                ann.loraine Ann Loraine made changes -
                Epic Link IGBF-1765 [ 17855 ]
                noor91zahara Noor Zahara (Inactive) made changes -
                Assignee Noor Zahara [ noor91zahara ]
                noor91zahara Noor Zahara (Inactive) made changes -
                Status Open [ 1 ] To-Do [ 10305 ]
                noor91zahara Noor Zahara (Inactive) made changes -
                Status To-Do [ 10305 ] In Progress [ 3 ]
                Hide
                Permalink
                noor91zahara Noor Zahara (Inactive) added a comment - - edited

                I tried replicating the issue using http://lorainelab-quickload.scidas.org/IGBF-1362 by changing the code in IGBTrustManager.java #checkServerTrusted to forcefully ask to trust the certificate.

                Findings

                1. If the user selects yes, then the data is getting loaded.
                2. User selects no, then an exception is thrown.

                Show
                noor91zahara Noor Zahara (Inactive) added a comment - - edited I tried replicating the issue using http://lorainelab-quickload.scidas.org/IGBF-1362 by changing the code in IGBTrustManager.java #checkServerTrusted to forcefully ask to trust the certificate. Findings 1. If the user selects yes, then the data is getting loaded. 2. User selects no, then an exception is thrown.
                Hide
                Permalink
                ann.loraine Ann Loraine added a comment -

                Could not reproduce.

                Show
                ann.loraine Ann Loraine added a comment - Could not reproduce.
                ann.loraine Ann Loraine made changes -
                Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
                ann.loraine Ann Loraine made changes -
                Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
                ann.loraine Ann Loraine made changes -
                Status First Level Review in Progress [ 10301 ] Needs 1st Level Review [ 10005 ]
                ann.loraine Ann Loraine made changes -
                Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
                ann.loraine Ann Loraine made changes -
                Status First Level Review in Progress [ 10301 ] Ready for Pull Request [ 10304 ]
                ann.loraine Ann Loraine made changes -
                Status Ready for Pull Request [ 10304 ] Pull Request Submitted [ 10101 ]
                ann.loraine Ann Loraine made changes -
                Status Pull Request Submitted [ 10101 ] Reviewing Pull Request [ 10303 ]
                ann.loraine Ann Loraine made changes -
                Status Reviewing Pull Request [ 10303 ] Merged Needs Testing [ 10002 ]
                ann.loraine Ann Loraine made changes -
                Status Merged Needs Testing [ 10002 ] Post-merge Testing In Progress [ 10003 ]
                ann.loraine Ann Loraine made changes -
                Resolution Done [ 10000 ]
                Status Post-merge Testing In Progress [ 10003 ] Closed [ 6 ]
                ann.loraine Ann Loraine made changes -
                Workflow Fall 2019 Workflow Update [ 20766 ] Revised Fall 2019 Workflow Update [ 22511 ]
                ann.loraine Ann Loraine made changes -
                Link This issue relates to IGBF-3128 [ IGBF-3128 ]

                  People

                  • Assignee:
                    noor91zahara Noor Zahara (Inactive)
                    Reporter:
                    nfreese Nowlan Freese
                  • Votes:
                    0 Vote for this issue
                    Watchers:
                    3 Start watching this issue

                    Dates

                    • Created:
                      Updated:
                      Resolved: