Project

General

Profile

Actions

Bug #15606

closed

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

Added by Peter Amstutz over 4 years ago. Updated about 4 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
-
Target version:
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 1 (0 open1 closed)

Task #15751: Review 15606-keep-web-errorsResolvedTom Clegg10/29/2019Actions

Related issues

Related to Arvados - Bug #15713: [Controller] Internal error not loggedResolvedTom Clegg10/24/2019Actions
Related to Arvados - Bug #15521: [keepstore] error reporting improvementsResolvedTom Clegg10/31/2019Actions
Actions #1

Updated by Peter Amstutz over 4 years ago

  • Status changed from New to In Progress
Actions #2

Updated by Peter Amstutz over 4 years ago

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

Updated by Tom Clegg over 4 years ago

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

Updated by Peter Amstutz over 4 years ago

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

Updated by Peter Amstutz over 4 years ago

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

Updated by Tom Clegg over 4 years ago

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

Updated by Tom Clegg over 4 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/

Actions #8

Updated by Lucas Di Pentima over 4 years ago

15606-keep-web-errors LGTM, thanks!

Actions #9

Updated by Tom Clegg over 4 years ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 100
Actions #10

Updated by Peter Amstutz about 4 years ago

  • Release set to 22
Actions

Also available in: Atom PDF