Bug #16774

keep-web needs to return user-visible errors

Added by Peter Amstutz about 2 months ago. Updated 1 day ago.

Status:
New
Priority:
Normal
Assigned To:
Category:
Keep
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:
(Total: 0.00 h)
Story points:
-

Description

I tried to download a file from keep-web with an invalid token. In the browser, without using developer tools, there is no feedback to the user why it didn't work.

  1. There is no error page. For any 4xx or 5xx error it should return a minimal error page
  2. Despite the error is actually that the token is invalid, it returns 404 instead of 401
  3. A 404 can mean either that an item doesn't exist, or that the user doesn't have permission to see it. The error text should reflect that.

Sort of related, I also realized that if keep-web sets a cookie with the API token, there's no way for the user to clear that cookie without going into browser settings.


Subtasks

Task #17027: ReviewNewNico C├ęsar


Related issues

Related to Arvados Epics - Story #16444: Improved error detection/reportingNew11/01/202011/30/2020

History

#1 Updated by Peter Amstutz about 2 months ago

  • Description updated (diff)

#2 Updated by Peter Amstutz about 2 months ago

  • Description updated (diff)

#3 Updated by Peter Amstutz about 2 months ago

  • Related to Story #16444: Improved error detection/reporting added

#4 Updated by Peter Amstutz about 1 month ago

  • Target version changed from 2020-09-23 Sprint to 2020-10-07 Sprint

#5 Updated by Peter Amstutz 29 days ago

  • Target version changed from 2020-10-07 Sprint to 2020-10-21 Sprint

#6 Updated by Peter Amstutz 15 days ago

  • Target version deleted (2020-10-21 Sprint)

#7 Updated by Peter Amstutz 7 days ago

  • Target version set to 2020-11-04 Sprint

#8 Updated by Peter Amstutz 7 days ago

  • Subject changed from keep-web should return better errors to keep-web needs to return user-visible errors

#9 Updated by Peter Amstutz 7 days ago

  • Description updated (diff)

#10 Updated by Peter Amstutz 3 days ago

  • Description updated (diff)

#11 Updated by Peter Amstutz 1 day ago

  • Category set to Keep

#12 Updated by Peter Amstutz 1 day ago

  • Assigned To set to Peter Amstutz

Also available in: Atom PDF