Bug #3115

[API] Retrieving a nonexistent keep disk as an admin should not create a new keep disk.

Added by Peter Amstutz almost 3 years ago. Updated 3 months ago.

Status:ResolvedStart date:
Priority:NormalDue date:
Assignee:-% Done:

100%

Category:-
Target version:-
Story points0.5
Velocity based estimate-

Description

  1. Log in as an administrator
  2. Go to keep_disks in work bench, e.g. https://workbench.4xphq.arvadosapi.com/keep_disks
  3. Go to one of the keep disk pages
  4. Go to the URL bar and change 1 character to create a new UUID that doesn't exist
  5. Instead of returning an error (e.g. UUID doesn't exist) it will create at least one new keep disk entry!

(Note: this is an API server bug, not a Workbench bug.)

Associated revisions

Revision 533a50bb
Added by Tom Clegg 3 months ago

Merge branch '3115-keep-disk-create-surprise'

closes #3115

History

#1 Updated by Peter Amstutz almost 3 years ago

  • Description updated (diff)

#2 Updated by Tom Clegg almost 3 years ago

  • Description updated (diff)

#3 Updated by Tom Clegg almost 3 years ago

  • Description updated (diff)
  • Subject changed from Visiting a nonexistent keep disk in workbench will create a new keep disk record instead of returning an error. to Retrieving a nonexistent keep disk will create a new keep disk record instead of returning an error.

#4 Updated by Tom Clegg almost 3 years ago

  • Target version set to Arvados Future Sprints

#5 Updated by Tom Clegg almost 3 years ago

  • Target version changed from Arvados Future Sprints to 2014-08-06 Sprint

#6 Updated by Tom Clegg almost 3 years ago

  • Story points set to 0.5

#7 Updated by Tom Clegg almost 3 years ago

  • Subject changed from Retrieving a nonexistent keep disk will create a new keep disk record instead of returning an error. to [API] Retrieving a nonexistent keep disk as an admin should not create a new keep disk.

#8 Updated by Ward Vandewege almost 3 years ago

  • Target version changed from 2014-08-06 Sprint to Arvados Future Sprints

#9 Updated by Tom Clegg 3 months ago

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

Applied in changeset arvados|commit:533a50bbcee4ece451a65e87066563a5b33ef150.

#10 Updated by Tom Clegg 3 months ago

  • Target version deleted (Arvados Future Sprints)

Also available in: Atom PDF