Bug #15606

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

Added by Peter Amstutz 11 months ago. Updated 7 months 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 9 months ago

Merge branch '15606-keep-web-errors'

closes #15606

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

History

#1 Updated by Peter Amstutz 11 months ago

  • Status changed from New to In Progress

#2 Updated by Peter Amstutz 11 months ago

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

#3 Updated by Tom Clegg 10 months ago

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

#4 Updated by Peter Amstutz 10 months ago

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

#5 Updated by Peter Amstutz 10 months ago

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

#6 Updated by Tom Clegg 10 months ago

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

#7 Updated by Tom Clegg 9 months 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 9 months ago

15606-keep-web-errors LGTM, thanks!

#9 Updated by Tom Clegg 9 months ago

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

#10 Updated by Peter Amstutz 7 months ago

  • Release set to 22

Also available in: Atom PDF