Project

General

Profile

Actions

Bug #15921

closed

[Workbench2] Remove spurious trailing slash from API endpoints

Added by Tom Clegg over 4 years ago. Updated over 4 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
Workbench2
Target version:
Story points:
-
Release relationship:
Auto

Description

For example, Workbench2 reportedly requests "POST /arvados/v1/collections/" when creating a new collection. The correct endpoint is "POST /arvados/v1/collections".


Subtasks 1 (0 open1 closed)

Task #15927: Review 15921-trailing-slash-fixResolvedPeter Amstutz12/10/2019Actions

Related issues

Related to Arvados - Bug #15916: [controller] POST /arvados/v1/collections/ (with extra trailing slash) returns 404RejectedActions
Actions #1

Updated by Tom Clegg over 4 years ago

  • Related to Bug #15916: [controller] POST /arvados/v1/collections/ (with extra trailing slash) returns 404 added
Actions #2

Updated by Peter Amstutz over 4 years ago

  • Assigned To set to Lucas Di Pentima
Actions #3

Updated by Lucas Di Pentima over 4 years ago

  • Status changed from New to In Progress
Actions #4

Updated by Lucas Di Pentima over 4 years ago

Fix at commit 75ba9a3a (wb2 repo) - branch 15921-trailing-slash-fix

Trailing slashes were being added to every resource. Fixed that and updated the necessary calls to add the missing slash when needed.

Actions #5

Updated by Lucas Di Pentima over 4 years ago

Update at commit a908c393 fixes the api call made by the the link account service.

Actions #6

Updated by Peter Amstutz over 4 years ago

This came up originally trying to create a collection (although other stuff would have failed too).

I tried to create a collection and ran into another bug.

I fixed the bug and pushed c055c90f752c104305e4097a9b7b5d029615e08f

This LGTM.

Actions #7

Updated by Lucas Di Pentima over 4 years ago

  • Status changed from In Progress to Resolved
Actions

Also available in: Atom PDF