diff --git a/src/console/user_api.rs b/src/console/user_api.rs
index 1b6b69f0..dd858a78 100644
--- a/src/console/user_api.rs
+++ b/src/console/user_api.rs
@@ -141,6 +141,12 @@ pub async fn add_user(
     web::Form(user_param): web::Form<UpdateUserInfoParam>,
 ) -> actix_web::Result<impl Responder> {
     let user: UserDto = user_param.into();
+    if user.roles.is_none() {
+        return Ok(HttpResponse::Ok().json(ApiResult::<()>::error(
+            "USER_ROLE_IS_EMPTY".to_string(),
+            Some("user roles is empty".to_owned()),
+        )));
+    }
     let msg = UserManagerReq::AddUser {
         user: UserDto {
             username: user.username,
diff --git a/src/console/v2/user_api.rs b/src/console/v2/user_api.rs
index f5f0ab61..56e204cd 100644
--- a/src/console/v2/user_api.rs
+++ b/src/console/v2/user_api.rs
@@ -85,6 +85,12 @@ pub async fn add_user(
     web::Json(user_param): web::Json<UpdateUserInfoParam>,
 ) -> actix_web::Result<impl Responder> {
     let user: UserDto = user_param.into();
+    if user.roles.is_none() {
+        return Ok(HttpResponse::Ok().json(ApiResult::<()>::error(
+            "USER_ROLE_IS_EMPTY".to_string(),
+            Some("user roles is empty".to_owned()),
+        )));
+    }
     let msg = UserManagerReq::AddUser {
         user: UserDto {
             username: user.username,