[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