search

silexlabs / cloud-explorer

*this version is deprecated, see CloudExplorer2* File manager for your users cloud services such as dropbox, google drive and FTP
http://cloud-explorer.org
52 stars 23 forks source link

bug when returning 401 errors #41

Closed lexoyo closed 9 years ago

lexoyo commented 10 years ago

Hi @zabojad :) Sometimes things go wrong between unifile and dropbox, and it results in a 401 which means that the user will have to re-login... It is not critical but that would be cool if a 401 could bring the "click here" popup..

zabojad commented 10 years ago

Hi there ! I'll see what I can do :) !

zabojad commented 10 years ago

Since fix of #42, error message from unifile are now displayed in error popups.

I do not think displaying the "click here" popup on every 401 is a good idea (it could be due to another reason). The best would be that on unifile side, the logic decides if we should ask the user to logout and login once again or not and tell it in the error message it sends back to CE...

Please close this issue if that's OK for you.

lexoyo commented 10 years ago

Well 401 means that you are not logged in anymore

zabojad commented 10 years ago

damn, that's true :) ! ok so I agree we should display the click here popup on every 401. I'll fix that soon.

lexoyo commented 10 years ago

bump ;)

zabojad commented 10 years ago

Don't worry, I have not forgotten... Will do in before end of the coming week.

zabojad commented 10 years ago

OK, I've pushed a fix for that in cdee79cb2a87f7f6b27f4234542aa57421dae921. Please tell me if this covers the cases where you've experienced that problem.

zabojad commented 9 years ago

ping :)

lexoyo commented 9 years ago

:+1: :sunny: