[ARVADOS] created: 2.1.0-2450-g9c3c0e489
Git user
git at public.arvados.org
Wed May 11 13:05:03 UTC 2022
at 9c3c0e489ddd52c1d46732cec5c87710243aac27 (commit)
commit 9c3c0e489ddd52c1d46732cec5c87710243aac27
Author: Lucas Di Pentima <lucas.dipentima at curii.com>
Date: Wed May 11 10:02:00 2022 -0300
19103: Fixes the default Limit param for ACA list requests.
Arvados-DCO-1.1-Signed-off-by: Lucas Di Pentima <lucas.dipentima at curii.com>
diff --git a/lib/controller/router/router.go b/lib/controller/router/router.go
index 05bdb4754..586ea8e67 100644
--- a/lib/controller/router/router.go
+++ b/lib/controller/router/router.go
@@ -407,7 +407,7 @@ func (rtr *router) addRoutes() {
},
{
arvados.EndpointAPIClientAuthorizationList,
- func() interface{} { return &arvados.ListOptions{} },
+ func() interface{} { return &arvados.ListOptions{Limit: -1} },
func(ctx context.Context, opts interface{}) (interface{}, error) {
return rtr.backend.APIClientAuthorizationList(ctx, *opts.(*arvados.ListOptions))
},
commit 7e02a7ba09642545ef5f567da5b18c91af7e3366
Author: Lucas Di Pentima <lucas.dipentima at curii.com>
Date: Wed May 11 09:48:48 2022 -0300
19103: Exposes the bug with a test.
Arvados-DCO-1.1-Signed-off-by: Lucas Di Pentima <lucas.dipentima at curii.com>
diff --git a/lib/controller/router/router_test.go b/lib/controller/router/router_test.go
index ce440dac5..11b090a21 100644
--- a/lib/controller/router/router_test.go
+++ b/lib/controller/router/router_test.go
@@ -92,6 +92,12 @@ func (s *RouterSuite) TestOptions(c *check.C) {
shouldCall: "CollectionList",
withOptions: arvados.ListOptions{Limit: -1},
},
+ {
+ method: "GET",
+ path: "/arvados/v1/api_client_authorizations",
+ shouldCall: "APIClientAuthorizationList",
+ withOptions: arvados.ListOptions{Limit: -1},
+ },
{
method: "GET",
path: "/arvados/v1/collections?limit=123&offset=456&include_trash=true&include_old_versions=1",
-----------------------------------------------------------------------
hooks/post-receive
--
More information about the arvados-commits
mailing list