Bug #15606

[keep-web] logging doesn't include error messages

Added by Peter Amstutz about 2 years ago. Updated almost 2 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
-
Target version:
Start date:
10/29/2019
Due date:
% Done:

100%

Estimated time:
(Total: 0.00 h)
Story points:
-
Release relationship:
Auto

Description

Community user on gitter is attempting to set up keep-web. It was first returning a 400 status code (which we determined could be worked around by setting TrustAllContent: true but there is no indication about why it was failing). After that it returned a 500 status code (still haven't figured it out). Keep-web needs to log error messages make it possible to diagnose failures.


Subtasks

Task #15751: Review 15606-keep-web-errorsResolvedTom Clegg


Related issues

Related to Arvados - Bug #15713: [Controller] Internal error not loggedResolved10/24/2019

Related to Arvados - Bug #15521: [keepstore] error reporting improvementsResolved10/31/2019

Associated revisions

Revision e5a8dfa1
Added by Tom Clegg almost 2 years ago

Merge branch '15606-keep-web-errors'

closes #15606

Arvados-DCO-1.1-Signed-off-by: Tom Clegg <>

History

#1 Updated by Peter Amstutz about 2 years ago

  • Status changed from New to In Progress

#2 Updated by Peter Amstutz about 2 years ago

  • Description updated (diff)
  • Status changed from In Progress to New

#3 Updated by Tom Clegg about 2 years ago

  • Target version changed from To Be Groomed to Arvados Future Sprints

#4 Updated by Peter Amstutz about 2 years ago

  • Related to Bug #15713: [Controller] Internal error not logged added

#5 Updated by Peter Amstutz about 2 years ago

  • Related to Bug #15521: [keepstore] error reporting improvements added

#6 Updated by Tom Clegg about 2 years ago

  • Assigned To set to Tom Clegg
  • Target version changed from Arvados Future Sprints to 2019-11-06 Sprint

#7 Updated by Tom Clegg almost 2 years ago

Updated a few error messages, including the "400 can't serve inline content because XSS" error, which was sending no text at all, and was almost certainly the one that motivated this issue.

15606-keep-web-errors @ fb3568c55a9f64a6027fcfe78ca21a79461ea348 -- https://ci.curoverse.com/view/Developer/job/developer-run-tests/1614/

#8 Updated by Lucas Di Pentima almost 2 years ago

15606-keep-web-errors LGTM, thanks!

#9 Updated by Tom Clegg almost 2 years ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 100

#10 Updated by Peter Amstutz almost 2 years ago

  • Release set to 22

Also available in: Atom PDF