sciserver / scidrive-ui

Web UI for scidrive
http://www.scidrive.org
Apache License 2.0
0 stars 0 forks source link

After Logging Out and Logging back in, was put in the previous "Share" View #30

Closed jkim485 closed 8 years ago

jkim485 commented 9 years ago

After Logging Out and Logging back in, was put in the previous "Share" View.

I should have started up in SciDrive in my "Root" folder, and worse still, I do not have access to my root folder.

We also need to add this as a test in the Test Script, under the section SD_WU_6.

jkim485 commented 9 years ago

More details, as this only happens in a specific circumstance.

  1. Start logged out of SciServer
  2. Start SciDrive app and Login.
  3. You will be in Root Dir
  4. Open a Share Then one of two things can happen: A. The BUG Case A.1 While the Share is open, Logout of SciDrive, and close the SciDrive and Share tabs A.2 Login to SCiserver through the Portal A.3 Open a tab and navigate to SciDrive A.4 "Login" (which you already are) A.5 BUG! Open the previous share

B. It Still Works Case B.1 While the Share is open, Logout of SciDrive, and close the SciDrive and Share tabs B.2 Open a tab and navigate to SciDrive B.3 "Login" - it will redirect to the Portal B.4 SciDrive opens in the correct Root folder.

jkim485 commented 9 years ago

Tested on scidrive 2 the following and a similar issue occurred as above. The steps taken are as follows: a new bug case C.1. Login to scidrive2 (on tab1) C.2 Share a folder C.3. open the share view.(on tab2) C.4. logout scidrive2 via sciserver (on tab1) C.5. open new tab and go to scidrive2 and click login. (tab3) C.6. Login via sci server login portal (tab3) C.7 shared view appears instead of root folder.(on tab3)

jkim485 commented 9 years ago

Tested on scidrive 2: repeated the steps A.1-A.5 - The issue has not been fixed.

jkim485 commented 9 years ago

Tested on scidrive 2: repeated the steps B.1-B.4 - Produced a bug - The step B.4: SciDrive opens in the shared folder.

@mikear69 Please check these steps to confirm whether there is a bug or not. Thanks.

dimm0 commented 9 years ago

Currently SciDrive stores the user session in browser storage. If user opens a share, this will be stored in browser and will be used on next visit, even if a new authentication is performed. To exit from share view, logoff button can be used. Alternatively, a share view can be only limited to a single visit and forgotten on tab close. Is that better?

jkim485 commented 8 years ago

Alternatively, a share view can be only limited to a single visit and forgotten on tab close. Is that better? Yes. @mikear69 please confirm it. YES

dimm0 commented 8 years ago

fixed