Project

General

Custom queries

Profile

Actions

Bug #6754

closed

[Workbench] Report HTTP error instead of sending 0-byte file when error fetching from keep

Added by Peter Amstutz over 9 years ago. Updated almost 8 years ago.

Status:
Rejected
Priority:
Normal
Assigned To:
-
Category:
Workbench
Target version:
Story points:
1.0

Description

Problem

Workbench shows files from collections by piping from arv-get. If arv-get can't download the file for any reason, Workbench returns a 200 OK status, with a 0-byte response.

Fix

  • Add an attribute to CollectionsController's FileStreamer class, and corresponding test stub classes, that captures the exit status of arv-get ($?). Before arv-get is run, this attribute is nil.
  • Near the end of CollectionsController#show_file, capture the FileStreamer created by file_enumerator. Hook up its each strings to response.stream as we do now. In the ensure block, if we haven't written any bytes to response.stream and the arv-get exit attribute does not have status 0, set response.status = 500.
#1

Updated by Peter Amstutz over 9 years ago

  • Category set to Workbench
#3

Updated by Brett Smith over 9 years ago

  • Target version changed from Bug Triage to Arvados Future Sprints
#4

Updated by Brett Smith over 9 years ago

  • Description updated (diff)
#5

Updated by Brett Smith over 9 years ago

  • Story points set to 1.0
#6

Updated by Brett Smith over 9 years ago

  • Target version changed from Arvados Future Sprints to Deferred
#7

Updated by Peter Amstutz almost 8 years ago

  • Status changed from New to Rejected
Actions

Also available in: Atom PDF