Fix error handling in GetUsers

A missing return statement caused GetUsers to return missleading results when the identity backend returned an error.
owncloud · Mar 22, 2022 · aab2d17 · aab2d17
1 parent 1265fd6
commit aab2d17
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/changelog/unreleased/graph-getusers-err.md b/changelog/unreleased/graph-getusers-err.md
@@ -0,0 +1,6 @@
+Bugfix: Fix error handling in GraphAPI GetUsers call
+
+A missing return statement caused GetUsers to return missleading results when
+the identity backend returned an error.
+
+https://github.com/owncloud/ocis/pull/3357
diff --git a/graph/pkg/service/v0/users.go b/graph/pkg/service/v0/users.go
@@ -46,6 +46,7 @@ func (g Graph) GetUsers(w http.ResponseWriter, r *http.Request) {
  } else {
  errorcode.GeneralException.Render(w, r, http.StatusInternalServerError, err.Error())
  }
+ return
  }
  render.Status(r, http.StatusOK)
  render.JSON(w, r, &listResponse{Value: users})