#654 fix it: NullPointException

vipshop · Dec 26, 2019 · b4a806b · b4a806b
1 parent bd9101f
commit b4a806b
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/...sole-api/src/main/java/com/vip/saturn/job/console/controller/gui/JobConfigController.java b/...sole-api/src/main/java/com/vip/saturn/job/console/controller/gui/JobConfigController.java
@@ -45,8 +45,9 @@ public SuccessResponseEntity updateJobConfig(final HttpServletRequest request,
 			@AuditParam("jobName") @PathVariable String jobName, UpdateJobConfigVo updateJobConfigVo)
 			throws SaturnJobConsoleException {
 		assertIsPermitted(PermissionKeys.jobUpdate, namespace);
-		if(updateJobConfigVo.getGroups().length() >= 255) {
-			throw new SaturnJobConsoleException("分组总数不能超过15个");
+		String groups = updateJobConfigVo.getGroups();
+		if (groups != null && groups.length() > 255) {
+			throw new SaturnJobConsoleException("分组过长，不能超过255个字符");
 		}
 		jobService.updateJobConfig(namespace, updateJobConfigVo.toJobConfig(), getCurrentLoginUserName());
 		return new SuccessResponseEntity();