17119: one more change after review feedback.

[arvados.git] / lib / controller / localdb / conn.go

diff --git a/lib/controller/localdb/conn.go b/lib/controller/localdb/conn.go
index c6b70ed2e98c7ec224a5aeea66d44611427ee62f..04f85cb5a9f54c2cd2286e33645fc4a62cb400ca 100644 (file)
--- a/lib/controller/localdb/conn.go
+++ b/lib/controller/localdb/conn.go
@@ -51,7 +51,7 @@ func (conn *Conn) UserAuthenticate(ctx context.Context, opts arvados.UserAuthent
 func (conn *Conn) GroupContents(ctx context.Context, options arvados.GroupContentsOptions) (arvados.ObjectList, error) {
        // The requested UUID can be a user (virtual home project), which we just pass on to
        // the API server.
-       if strings.Index(options.UUID, "j7d0g") != 6 {
+       if strings.Index(options.UUID, "-j7d0g-") != 5 {
                return conn.railsProxy.GroupContents(ctx, options)
        }
 
@@ -86,6 +86,8 @@ func (conn *Conn) GroupContents(ctx context.Context, options arvados.GroupConten
                                filter.Operand = tmp[2]
                                options.Filters = append(options.Filters, filter)
                        }
+               } else {
+                       return resp, fmt.Errorf("filter unparsable: not an array\n")
                }
                // Use the generic /groups/contents endpoint for filter groups
                options.UUID = ""