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

Fix the error where user is logged out when using search field

    Details

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

      Description

      When using the search field in Bioviz Connect, the user gets logged out after showing the ouroboros loading symbol for a few seconds. Cyverse Search APIs returns a 200 OK on Postman.

        Attachments

          Activity

          karthik Karthik Raveendran created issue -
          karthik Karthik Raveendran made changes -
          Field Original Value New Value
          Epic Link IGBF-2376 [ 18533 ]
          karthik Karthik Raveendran made changes -
          Sprint Summer 2 [ 196 ] Summer 1 [ 195 ]
          karthik Karthik Raveendran made changes -
          Sprint Summer 1 [ 195 ] Summer 2 [ 196 ]
          karthik Karthik Raveendran made changes -
          Sprint Summer 2 [ 196 ] Summer 1 [ 195 ]
          karthik Karthik Raveendran made changes -
          Status To-Do [ 10305 ] In Progress [ 3 ]
          karthik Karthik Raveendran made changes -
          Sprint Summer 1 [ 195 ] Summer 2 [ 196 ]
          karthik Karthik Raveendran made changes -
          Summary Fix user logging out when using search issue Fix the error where user is logged out when using search field
          Hide
          karthik Karthik Raveendran added a comment -

          The response JSON from the search API request has been changed by Cyverse. After the user searches for something, we need to list all the files and folders with name similar to the search key. We also need to group it as folders and files and for that we use the document type value in the response JSON. Before, this value had the key, '_type', now, 'doc_type'. Secondly, the position of this key-value pair has changed.

          # Before:
          response_json['hits'][0]['_type']
          
          ## where the response_json, as the name suggests, is the response we get. 'hits' is the key for the list in that json with all the folders and files listed.
          
          # Now its changed to
          response_json['hits'][0]['_source']['doc_type']
          

          See commit

          Show
          karthik Karthik Raveendran added a comment - The response JSON from the search API request has been changed by Cyverse. After the user searches for something, we need to list all the files and folders with name similar to the search key. We also need to group it as folders and files and for that we use the document type value in the response JSON. Before, this value had the key, '_type', now, 'doc_type'. Secondly, the position of this key-value pair has changed. # Before: response_json['hits'][0]['_type'] ## where the response_json, as the name suggests, is the response we get. 'hits' is the key for the list in that json with all the folders and files listed. # Now its changed to response_json['hits'][0]['_source']['doc_type'] See commit
          karthik Karthik Raveendran made changes -
          Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
          karthik Karthik Raveendran made changes -
          Assignee Karthik Raveendran [ karthik ]
          Hide
          karthik Karthik Raveendran added a comment -

          To Test:

          • Pull the branch and run Bioviz Connect in your local system
          • Add a search text (eg. '.bam', 'fasta') in the search bar on top of the page
          • Click on the magnifying glass button next to the search bar
          • If search results appear as expected, the solution works
          • Click on top of the titles of the columns of the table to sort
          • If the table sorts as expected, the solution works
          Show
          karthik Karthik Raveendran added a comment - To Test: Pull the branch and run Bioviz Connect in your local system Add a search text (eg. '.bam', 'fasta') in the search bar on top of the page Click on the magnifying glass button next to the search bar If search results appear as expected, the solution works Click on top of the titles of the columns of the table to sort If the table sorts as expected, the solution works
          ann.loraine Ann Loraine made changes -
          Sprint Summer 2 [ 196 ] Summer 2, Summer 3 [ 196, 197 ]
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          nfreese Nowlan Freese made changes -
          Assignee Nowlan Freese [ nfreese ]
          Show
          karthik Karthik Raveendran added a comment - PR Submitted. https://bitbucket.org/nfreese/bioviz-connect/pull-requests/292
          karthik Karthik Raveendran made changes -
          Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
          karthik Karthik Raveendran made changes -
          Status First Level Review in Progress [ 10301 ] Ready for Pull Request [ 10304 ]
          karthik Karthik Raveendran made changes -
          Status Ready for Pull Request [ 10304 ] Pull Request Submitted [ 10101 ]
          Hide
          nfreese Nowlan Freese added a comment -

          PR merged

          Ann Loraine - I am unable to connect to the bvcprod2 instance either with the bioviz-connect-playbooks or through ssh. I was able to use the ansible playbooks a few months ago (see my notes on IGBF-3550), so I'm not sure what has changed.

          Show
          nfreese Nowlan Freese added a comment - PR merged Ann Loraine - I am unable to connect to the bvcprod2 instance either with the bioviz-connect-playbooks or through ssh. I was able to use the ansible playbooks a few months ago (see my notes on IGBF-3550 ), so I'm not sure what has changed.
          nfreese Nowlan Freese made changes -
          Status Pull Request Submitted [ 10101 ] Reviewing Pull Request [ 10303 ]
          nfreese Nowlan Freese made changes -
          Status Reviewing Pull Request [ 10303 ] Merged Needs Testing [ 10002 ]
          ann.loraine Ann Loraine made changes -
          Sprint Summer 2, Summer 3 [ 196, 197 ] Summer 2, Summer 3, Summer 4 [ 196, 197, 198 ]
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          Hide
          ann.loraine Ann Loraine added a comment -

          Assigning this to me since re-activating the testing host is something I will need to do before Nowlan Freese can proceed with testing.

          Also, I would like to move this to the next sprint when I will have time to do the necessary work.

          Show
          ann.loraine Ann Loraine added a comment - Assigning this to me since re-activating the testing host is something I will need to do before Nowlan Freese can proceed with testing. Also, I would like to move this to the next sprint when I will have time to do the necessary work.
          ann.loraine Ann Loraine made changes -
          Assignee Nowlan Freese [ nfreese ] Ann Loraine [ aloraine ]
          ann.loraine Ann Loraine made changes -
          Sprint Summer 2, Summer 3, Summer 4 [ 196, 197, 198 ] Summer 2, Summer 3, Summer 5 [ 196, 197, 199 ]
          ann.loraine Ann Loraine made changes -
          Assignee Ann Loraine [ aloraine ] Nowlan Freese [ nfreese ]
          Hide
          ann.loraine Ann Loraine added a comment -

          As per discussion during scrum today, re-assigning to NF for him to deploy the new code and the new SSL certificates to the production site when ready.

          Show
          ann.loraine Ann Loraine added a comment - As per discussion during scrum today, re-assigning to NF for him to deploy the new code and the new SSL certificates to the production site when ready.
          ann.loraine Ann Loraine made changes -
          Sprint Summer 2, Summer 3, Summer 5 [ 196, 197, 199 ] Summer 2, Summer 3, Summer 5, Summer 6 [ 196, 197, 199, 200 ]
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          nfreese Nowlan Freese made changes -
          Status Merged Needs Testing [ 10002 ] Post-merge Testing In Progress [ 10003 ]
          Hide
          nfreese Nowlan Freese added a comment -

          Changes have been pushed to BioViz Connect.

          Deployed following instructions from bioviz-connect-playbooks repo with:

          ansible-playbook -i inventory.ini setup.yml

          See my comment in IGBF-3402 on how to delegate such that the sync_app task can be run correctly. Otherwise, remove the sync_app role from setup.yml and manually call the sync app endpoint:

          https://connect.bioviz.org/SyncAppDataToBioViz/?accesstoken=<validKeyCloakAccessToken>
          

          Testing:
          Searching in BioViz Connect for ".bam" returns files that end in .bam. I am able to click the View in IGB button and the data load in IGB.

          Closing ticket.

          Show
          nfreese Nowlan Freese added a comment - Changes have been pushed to BioViz Connect. Deployed following instructions from bioviz-connect-playbooks repo with: ansible-playbook -i inventory.ini setup.yml See my comment in IGBF-3402 on how to delegate such that the sync_app task can be run correctly. Otherwise, remove the sync_app role from setup.yml and manually call the sync app endpoint: https: //connect.bioviz.org/SyncAppDataToBioViz/?accesstoken=<validKeyCloakAccessToken> Testing: Searching in BioViz Connect for ".bam" returns files that end in .bam. I am able to click the View in IGB button and the data load in IGB. Closing ticket.
          nfreese Nowlan Freese made changes -
          Assignee Nowlan Freese [ nfreese ] Karthik Raveendran [ karthik ]
          nfreese Nowlan Freese made changes -
          Resolution Done [ 10000 ]
          Status Post-merge Testing In Progress [ 10003 ] Closed [ 6 ]
          nfreese Nowlan Freese made changes -
          Story Points 0.5 1

            People

            • Assignee:
              karthik Karthik Raveendran
              Reporter:
              karthik Karthik Raveendran
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: