Merge pull request goharbor#7062 from jwangyangls/userListPermission

fix get users list about permission issue
zhangg · Mar 4, 2019 · 685e590 · 685e590
2 parents 62bb10d + 1e17036
commit 685e590
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/src/portal/src/app/project/member/add-member/add-member.component.ts b/src/portal/src/app/project/member/add-member/add-member.component.ts
@@ -94,7 +94,7 @@ export class AddMemberComponent implements AfterViewChecked, OnInit, OnDestroy {
       hasProjectAdminRole = (<Project>resolverData['projectResolver']).has_project_admin_role;
     }
     if (hasProjectAdminRole) {
-      this.userService.getUsers()
+      this.userService.getUsersNameList()
         .then(users => {
           this.userLists = users;
         });

diff --git a/src/portal/src/app/user/user.service.ts b/src/portal/src/app/user/user.service.ts
@@ -20,6 +20,7 @@ import { User, LDAPUser } from './user';
 import LDAPUsertoUser from './user';
 
 const userMgmtEndpoint = '/api/users';
+const userListSearch = '/api/users/search';
 const ldapUserEndpoint = '/api/ldap/users';
 
 /**
@@ -39,6 +40,11 @@ export class UserService {
     }
 
     // Get the user list
+    getUsersNameList(): Promise<User[]> {
+        return this.http.get(userListSearch, HTTP_GET_OPTIONS).toPromise()
+            .then(response => response.json() as User[])
+            .catch(error => this.handleError(error));
+    }
     getUsers(): Promise<User[]> {
         return this.http.get(userMgmtEndpoint, HTTP_GET_OPTIONS).toPromise()
             .then(response => response.json() as User[])