fix updateGroupUserList api issue

src/rest-server/docs/swagger.yaml

@@ -725,6 +725,8 @@ paths:
  description: description
  externalName: externalName
  extension: {}
+ "401":
+ $ref: "#/components/responses/UnauthorizedUserError"
  "500":
  $ref: "#/components/responses/UnknownError"
  post:
@@ -930,6 +932,8 @@ paths:
  clusterAdmin: false
  "401":
  $ref: "#/components/responses/UnauthorizedUserError"
+ "403":
+ $ref: "#/components/responses/ForbiddenUserError"
  "500":
  $ref: "#/components/responses/UnknownError"
  /api/v2/virtual-clusters:

src/rest-server/src/controllers/v2/group.js

@@ -36,7 +36,8 @@ const getGroup = async (req, res, next) => {
  ),
  );
  }
- return next(createError.unknown(error));
+ const message = error.data ? error.data.reason : error;
+ return next(createError.unknown(message));
  }
 };
 
@@ -45,7 +46,8 @@ const getAllGroup = async (req, res, next) => {
  const groupList = await groupModel.getAllGroup();
  return res.status(200).json(groupList);
  } catch (error) {
- return next(createError.unknown(error));
+ const message = error.data ? error.data.reason : error;
+ return next(createError.unknown(message));
  }
 };
 
@@ -64,7 +66,8 @@ const getGroupUserList = async (req, res, next) => {
  }
  return res.status(200).json(userlist);
  } catch (error) {
- return next(createError.unknown(error));
+ const message = error.data ? error.data.reason : error;
+ return next(createError.unknown(message));
  }
 };
 
@@ -82,7 +85,8 @@ const createGroup = async (req, res, next) => {
  message: 'group is created successfully',
  });
  } catch (error) {
- return next(createError.unknown(error));
+ const message = error.data ? error.data.reason : error;
+ return next(createError.unknown(message));
  }
 };
 
@@ -123,7 +127,8 @@ const updateGroup = async (req, res, next) => {
  ),
  );
  }
- return next(createError.unknown(error));
+ const message = error.data ? error.data.reason : error;
+ return next(createError.unknown(message));
  }
 };
 
@@ -135,7 +140,8 @@ const deleteGroup = async (req, res, next) => {
  message: 'group is removed successfully',
  });
  } catch (error) {
- return next(createError.unknown(error));
+ const message = error.data ? error.data.reason : error;
+ return next(createError.unknown(message));
  }
 };
 

src/rest-server/src/controllers/v2/user.js

@@ -353,7 +353,7 @@ const updateUserGroupList = async (req, res, next) => {
  let userValue;
  try {
  userValue = await userModel.getUser(username);
- userValue.grouplist = await updateGroupListInternal(userValue.grouplist);
+ userValue.grouplist = await updateGroupListInternal(req.body.grouplist);
  } catch (error) {
  if (error.code === 'NoGroupError') {
  return next(error);