Bug #17111

unsigned_manifest_text being returned in objectList (contents?) even when not selected

Added by Peter Amstutz 18 days ago. Updated 14 days ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
API
Target version:
Start date:
11/13/2020
Due date:
% Done:

100%

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

Description

The 'contents' endpoint is returning "unsigned_manifest_text" when is potentially a very large field. This is inconsistent with the behavior of the 'collections' listing and not intended to happen unless explicitly selected for.


Subtasks

Task #17113: ReviewResolvedPeter Amstutz

Associated revisions

Revision 059e2806
Added by Tom Clegg 17 days ago

Merge branch '17111-deselect-unsigned-manifest'

fixes #17111

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

History

#1 Updated by Peter Amstutz 17 days ago

  • Description updated (diff)

#2 Updated by Peter Amstutz 17 days ago

  • Assigned To set to Tom Clegg

#3 Updated by Tom Clegg 17 days ago

  • Status changed from New to In Progress

#4 Updated by Tom Clegg 17 days ago

Indeed, groups#contents was automatically deselecting manifest_text, but not unsigned_manifest_text.

17111-deselect-unsigned-manifest @ 2d98a947ca12fa6b1391e8b882bbcfbe43970ee3 -- https://ci.arvados.org/view/Developer/job/developer-run-tests/2173/

#5 Updated by Peter Amstutz 17 days ago

  • Release set to 36

#6 Updated by Peter Amstutz 17 days ago

Tom Clegg wrote:

Indeed, groups#contents was automatically deselecting manifest_text, but not unsigned_manifest_text.

17111-deselect-unsigned-manifest @ 2d98a947ca12fa6b1391e8b882bbcfbe43970ee3 -- https://ci.arvados.org/view/Developer/job/developer-run-tests/2173/

The fix makes sense and the tests pass so this LGTM.

#7 Updated by Anonymous 17 days ago

  • Status changed from In Progress to Resolved

Also available in: Atom PDF