From 67dba8c8facdb5b48a6f58d18134ca411ae36683 Mon Sep 17 00:00:00 2001 From: Andy Wang Date: Wed, 26 Jan 2022 00:57:18 +0000 Subject: [PATCH] Return empty slices instead of nil for ListAllUsers --- internal/server/usermanager/localmanager.go | 3 +++ internal/server/usermanager/voidmanager.go | 2 +- 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/internal/server/usermanager/localmanager.go b/internal/server/usermanager/localmanager.go index d60f62e..4b3096b 100644 --- a/internal/server/usermanager/localmanager.go +++ b/internal/server/usermanager/localmanager.go @@ -191,6 +191,9 @@ func (manager *localManager) ListAllUsers() (infos []UserInfo, err error) { }) return err }) + if infos == nil { + infos = []UserInfo{} + } return } diff --git a/internal/server/usermanager/voidmanager.go b/internal/server/usermanager/voidmanager.go index a20ab3c..d317353 100644 --- a/internal/server/usermanager/voidmanager.go +++ b/internal/server/usermanager/voidmanager.go @@ -15,7 +15,7 @@ func (v *Voidmanager) UploadStatus(updates []StatusUpdate) ([]StatusResponse, er } func (v *Voidmanager) ListAllUsers() ([]UserInfo, error) { - return nil, ErrMangerIsVoid + return []UserInfo{}, ErrMangerIsVoid } func (v *Voidmanager) GetUserInfo(UID []byte) (UserInfo, error) {