From 9da0f151682dda50f9f76e2dbdcded7150d7275b Mon Sep 17 00:00:00 2001 From: Gianmaria Del Monte <39946305+gmgigi96@users.noreply.github.com> Date: Thu, 11 Nov 2021 12:20:26 +0100 Subject: [PATCH] Fix nil pointer exception when resolving members of a group (rest driver) (#2265) --- changelog/unreleased/fix-nil-exception-groups.md | 3 +++ pkg/cbox/group/rest/rest.go | 4 +++- 2 files changed, 6 insertions(+), 1 deletion(-) create mode 100644 changelog/unreleased/fix-nil-exception-groups.md diff --git a/changelog/unreleased/fix-nil-exception-groups.md b/changelog/unreleased/fix-nil-exception-groups.md new file mode 100644 index 0000000000..5c773e529f --- /dev/null +++ b/changelog/unreleased/fix-nil-exception-groups.md @@ -0,0 +1,3 @@ +Bugfix: Fix nil pointer exception when resolving members of a group (rest driver) + +https://github.com/cs3org/reva/pull/2265 \ No newline at end of file diff --git a/pkg/cbox/group/rest/rest.go b/pkg/cbox/group/rest/rest.go index 2e93809d60..439ec9c6c9 100644 --- a/pkg/cbox/group/rest/rest.go +++ b/pkg/cbox/group/rest/rest.go @@ -346,7 +346,9 @@ func (m *manager) GetMembers(ctx context.Context, gid *grouppb.GroupId) ([]*user if !ok { return nil, errors.New("rest: error in type assertion") } - users = append(users, &userpb.UserId{OpaqueId: userInfo["upn"].(string), Idp: m.conf.IDProvider}) + if id, ok := userInfo["upn"].(string); ok { + users = append(users, &userpb.UserId{OpaqueId: id, Idp: m.conf.IDProvider}) + } } if err = m.cacheGroupMembers(gid, users); err != nil {