From 9ec895d7754fdfd4918fab69cc220c0d814c41f7 Mon Sep 17 00:00:00 2001 From: jinpengyong <jpy123456> Date: Tue, 23 Mar 2021 17:36:23 +0800 Subject: [PATCH] 删除,修改用户更新redis --- screen-api/src/main/java/com/moral/api/config/Interceptor/WebAppConfiguration.java | 2 screen-api/src/main/java/com/moral/api/controller/LoginController.java | 41 ++++++++++++-------- screen-api/src/main/java/com/moral/api/service/impl/UserServiceImpl.java | 43 ++++++++++++--------- screen-common/src/main/java/com/moral/util/TokenUtils.java | 9 ++++ screen-api/src/main/java/com/moral/api/controller/UserController.java | 22 ++-------- pom.xml | 2 6 files changed, 65 insertions(+), 54 deletions(-) diff --git a/pom.xml b/pom.xml index fd7b9a2..be47e13 100644 --- a/pom.xml +++ b/pom.xml @@ -108,7 +108,7 @@ <scope>test</scope> </dependency> <dependency> - <groupId>com.github.penggle</groupId> + <groupId>com.google.code.kaptcha</groupId> <artifactId>kaptcha</artifactId> <version>2.3.2</version> </dependency> diff --git a/screen-api/src/main/java/com/moral/api/config/Interceptor/WebAppConfiguration.java b/screen-api/src/main/java/com/moral/api/config/Interceptor/WebAppConfiguration.java index 99e60ac..836ef67 100644 --- a/screen-api/src/main/java/com/moral/api/config/Interceptor/WebAppConfiguration.java +++ b/screen-api/src/main/java/com/moral/api/config/Interceptor/WebAppConfiguration.java @@ -13,6 +13,6 @@ public void addInterceptors(InterceptorRegistry registry) { registry.addInterceptor(new AuthenticationInterceptor()) .addPathPatterns("/**") - .excludePathPatterns("/**/login/**", "/**/logout/**", "/swagger-ui.html/**", "/swagger-resources/**"); + .excludePathPatterns("/**/login/**", "/**/logout/**", "/swagger-ui.html/**", "/swagger-resources/**","/**/verificationCode/**","/**/user/**"); } } diff --git a/screen-api/src/main/java/com/moral/api/controller/LoginController.java b/screen-api/src/main/java/com/moral/api/controller/LoginController.java index faf1d3b..cb1862a 100644 --- a/screen-api/src/main/java/com/moral/api/controller/LoginController.java +++ b/screen-api/src/main/java/com/moral/api/controller/LoginController.java @@ -10,21 +10,25 @@ import java.util.Map; import javax.servlet.http.HttpServletRequest; +import javax.servlet.http.HttpServletResponse; import org.springframework.beans.factory.annotation.Autowired; import org.springframework.util.ObjectUtils; +import org.springframework.web.bind.annotation.RequestBody; import org.springframework.web.bind.annotation.RequestMapping; import org.springframework.web.bind.annotation.RequestMethod; import org.springframework.web.bind.annotation.RestController; +import com.baomidou.mybatisplus.core.conditions.query.QueryWrapper; +import com.moral.api.entity.SysConfig; import com.moral.api.service.SysConfigService; import com.moral.api.service.UserService; +import com.moral.constant.Constants; import com.moral.constant.ResponseCodeEnum; import com.moral.constant.ResultMessage; import com.moral.pojo.VerificationCode; import com.moral.util.KaptchaUtils; import com.moral.util.TokenUtils; -import com.moral.util.WebUtils; @Slf4j @Api(tags = {"������"}) @@ -38,13 +42,8 @@ private SysConfigService sysConfigService; @ApiOperation(value = "������", notes = "������") - @ApiImplicitParams({ - @ApiImplicitParam(name = "account", value = "������", required = true, paramType = "query", dataType = "String"), - @ApiImplicitParam(name = "password", value = "������", required = true, paramType = "query", dataType = "String") - }) @RequestMapping(value = "login", method = RequestMethod.POST) - public ResultMessage login(HttpServletRequest request) { - Map<String, Object> parameters = WebUtils.getParametersStartingWith(request, null); + public ResultMessage login(@RequestBody Map<String, Object> parameters) { if (!(parameters.containsKey("account") && parameters.containsKey("password"))) { return ResultMessage.fail(ResponseCodeEnum.PARAMETERS_IS_MISSING.getCode(), ResponseCodeEnum.PARAMETERS_IS_MISSING.getMsg()); } @@ -55,20 +54,18 @@ return ResultMessage.ok(result.get("data")); } - @ApiOperation(value = "������", notes = "������") + @ApiOperation(value = "������", notes = "������") @RequestMapping(value = "logout", method = RequestMethod.POST) @ApiImplicitParams({ - @ApiImplicitParam(name = "uid", value = "������", required = false, paramType = "query", dataType = "String"), + @ApiImplicitParam(name = "uid", value = "������id", required = true, paramType = "query", dataType = "String"), @ApiImplicitParam(name = "token", value = "token", required = true, paramType = "header", dataType = "String") }) - public ResultMessage logout(HttpServletRequest request) { - Map<String, Object> parameters = WebUtils.getParametersStartingWith(request, null); - if (!parameters.containsKey("uid")) { + public ResultMessage logout(String uid, HttpServletRequest request) { + if (uid == null) { return ResultMessage.fail(ResponseCodeEnum.PARAMETERS_IS_MISSING.getCode(), ResponseCodeEnum.PARAMETERS_IS_MISSING.getMsg()); } - String userId = parameters.get("uid").toString(); String token = request.getHeader("token"); - TokenUtils.destoryToken(userId, token); + TokenUtils.destoryToken(uid, token); return ResultMessage.ok(); } @@ -81,9 +78,19 @@ */ @ApiOperation(value = "���������������������������", notes = "���������������������������") - @RequestMapping(value = "verificationCodeConfig", method = RequestMethod.GET) - public ResultMessage verifyConfig() { - return null; + @RequestMapping(value = "verificationCode/config", method = RequestMethod.GET) + public void verifyConfig(HttpServletResponse response) { + QueryWrapper<SysConfig> queryWrapper = new QueryWrapper<>(); + queryWrapper.eq("type", Constants.VERIFICATION_TYPE); + SysConfig sysConfig = sysConfigService.getOne(queryWrapper); + String code = sysConfig.getCode(); + if (Constants.VERIFICATION_OPEN_CODE.equals(code)) { + try { + response.sendRedirect("/verificationCode/get"); + } catch (IOException e) { + e.printStackTrace(); + } + } } @ApiOperation(value = "���������������", notes = "���������������") diff --git a/screen-api/src/main/java/com/moral/api/controller/UserController.java b/screen-api/src/main/java/com/moral/api/controller/UserController.java index 6044b15..41e3d80 100644 --- a/screen-api/src/main/java/com/moral/api/controller/UserController.java +++ b/screen-api/src/main/java/com/moral/api/controller/UserController.java @@ -11,6 +11,7 @@ import javax.servlet.http.HttpServletRequest; import org.springframework.beans.factory.annotation.Autowired; +import org.springframework.web.bind.annotation.RequestBody; import org.springframework.web.bind.annotation.RequestMapping; import org.springframework.web.bind.annotation.RequestMethod; import org.springframework.web.bind.annotation.RestController; @@ -35,16 +36,10 @@ @ApiOperation(value = "������������", notes = "������������") @ApiImplicitParams({ - @ApiImplicitParam(name = "account", value = "������,������6-10", required = true, paramType = "query", dataType = "String"), - @ApiImplicitParam(name = "password", value = "������,������6-20", required = true, paramType = "query", dataType = "String"), - @ApiImplicitParam(name = "userName", value = "������������", required = false, paramType = "query", dataType = "String"), - @ApiImplicitParam(name = "email", value = "������,������123456@qq.com", required = false, paramType = "query", dataType = "String"), - @ApiImplicitParam(name = "mobile", value = "���������,1������11���������", required = false, paramType = "query", dataType = "String"), - @ApiImplicitParam(name = "wechat", value = "������", required = false, paramType = "query", dataType = "String"), @ApiImplicitParam(name = "token", value = "token", required = true, paramType = "header", dataType = "String") }) @RequestMapping(value = "addUser", method = RequestMethod.POST) - public ResultMessage addUser(User user, HttpServletRequest request) { + public ResultMessage addUser(@RequestBody User user, HttpServletRequest request) { if (user.getAccount().isEmpty() || user.getPassword().isEmpty()) { return ResultMessage.fail(ResponseCodeEnum.PARAMETERS_IS_MISSING.getCode(), ResponseCodeEnum.PARAMETERS_IS_MISSING.getMsg()); @@ -59,8 +54,7 @@ @ApiOperation(value = "������������", notes = "������������") @ApiImplicitParams({ - @ApiImplicitParam(name = "userId", value = "������id", required = true, paramType = "query", dataType = "String"), - @ApiImplicitParam(name = "token", value = "token", required = true, paramType = "header", dataType = "String") + @ApiImplicitParam(name = "userId", value = "������id", required = true, paramType = "query", dataType = "String") }) @RequestMapping(value = "deleteUser", method = RequestMethod.POST) public ResultMessage deleteUser(String userId) { @@ -79,17 +73,11 @@ @ApiOperation(value = "������������������", notes = "������������������") @ApiImplicitParams({ - @ApiImplicitParam(name = "account", value = "������,������6-10", required = true, paramType = "query", dataType = "String"), - @ApiImplicitParam(name = "password", value = "������,������6-20", required = true, paramType = "query", dataType = "String"), - @ApiImplicitParam(name = "userName", value = "������������", required = false, paramType = "query", dataType = "String"), - @ApiImplicitParam(name = "email", value = "������,������123456@qq.com", required = false, paramType = "query", dataType = "String"), - @ApiImplicitParam(name = "mobile", value = "���������,1������11���������", required = false, paramType = "query", dataType = "String"), - @ApiImplicitParam(name = "wechat", value = "������", required = false, paramType = "query", dataType = "String"), @ApiImplicitParam(name = "token", value = "token", required = true, paramType = "header", dataType = "String") }) @RequestMapping(value = "updateUser", method = RequestMethod.POST) - public ResultMessage updateUser(User user, HttpServletRequest request) { - if (user.getAccount().isEmpty() || user.getPassword().isEmpty()) { + public ResultMessage updateUser(@RequestBody User user, HttpServletRequest request) { + if (user.getId() == null) { return ResultMessage.fail(ResponseCodeEnum.PARAMETERS_IS_MISSING.getCode(), ResponseCodeEnum.PARAMETERS_IS_MISSING.getMsg()); } diff --git a/screen-api/src/main/java/com/moral/api/service/impl/UserServiceImpl.java b/screen-api/src/main/java/com/moral/api/service/impl/UserServiceImpl.java index 2ee483f..8263599 100644 --- a/screen-api/src/main/java/com/moral/api/service/impl/UserServiceImpl.java +++ b/screen-api/src/main/java/com/moral/api/service/impl/UserServiceImpl.java @@ -115,7 +115,7 @@ userInfo.put("locationLevel", organization.getLocationLevel()); userInfo.put("groups", groups); Map<String, Object> map = new HashMap<>(); - map.put("userId",userBo.getId()); + map.put("userId", userBo.getId()); userInfo.putAll(getMenus(map)); try { //������token������������redis @@ -247,12 +247,18 @@ @Override @Transactional public void deleteUser(User user) { + String userId = user.getId().toString(); //������������user user.setIsDelete(Constants.DELETE); userMapper.updateById(user); //������������user_group UpdateWrapper<UserGroup> deleteWrapper = new UpdateWrapper<>(); - deleteWrapper.set("is_delete", Constants.DELETE).eq("user_id", user.getId()); + deleteWrapper.set("is_delete", Constants.DELETE).eq("user_id", userId); + //������redis + if (TokenUtils.hHasKey(userId)) { + String token = TokenUtils.hget(userId).toString(); + TokenUtils.destoryToken(userId, token); + } userGroupMapper.update(null, deleteWrapper); } @@ -265,7 +271,7 @@ queryWrapper.ne("id", user.getId()) .eq("account", user.getAccount()) .eq("is_delete", Constants.NOT_DELETE); - //������������������������ + //��������������������������� if (userMapper.selectOne(queryWrapper) != null) { result.put("code", ResponseCodeEnum.ACCOUNT_EXIST.getCode()); result.put("msg", ResponseCodeEnum.ACCOUNT_EXIST.getMsg()); @@ -285,12 +291,14 @@ } //������������������ String password = user.getPassword(); - //������������ - //password = AESUtils.decrypt(password, AESKey); - if (!RegexUtils.checkPassword(password)) { - result.put("code", ResponseCodeEnum.PASSWORD_INVALID.getCode()); - result.put("msg", ResponseCodeEnum.PASSWORD_INVALID.getMsg()); - return result; + if (password != null) { + //������������ + //password = AESUtils.decrypt(password, AESKey); + if (!RegexUtils.checkPassword(password)) { + result.put("code", ResponseCodeEnum.PASSWORD_INVALID.getCode()); + result.put("msg", ResponseCodeEnum.PASSWORD_INVALID.getMsg()); + return result; + } } //��������������������� if (user.getMobile() != null && !RegexUtils.checkMobile(user.getMobile())) { @@ -298,16 +306,22 @@ result.put("msg", ResponseCodeEnum.MOBILE_INVALID.getMsg()); return result; } - //������������ + //������MD5������ user.setPassword(MD5Utils.saltMD5(password)); user.setOrganizationId((int) currentUserInfo.get("organizationId")); user.setIsAdmin(false); - //��������������������������� + //������������������������������ Date userExpireTime = user.getExpireTime(); //��������������������������� Date currentUserExpireTime = (Date) currentUserInfo.get("expireTime"); if (userExpireTime == null || userExpireTime.getTime() > currentUserExpireTime.getTime()) { user.setExpireTime(currentUserExpireTime); + } + //������redis + String userId = user.getId().toString(); + if (TokenUtils.hHasKey(userId)) { + String deleterToken = TokenUtils.hget(userId).toString(); + TokenUtils.destoryToken(userId, deleterToken); } userMapper.updateById(user); return result; @@ -341,13 +355,6 @@ } if (wechat != null) { queryWrapper.like("wechat", wechat); - } - //������������ - if (!parameters.containsKey("page")) { - parameters.put("page", 0); - } - if (!parameters.containsKey("size")) { - parameters.put("size", 10); } int page = Integer.parseInt(parameters.get("page").toString()); diff --git a/screen-common/src/main/java/com/moral/util/TokenUtils.java b/screen-common/src/main/java/com/moral/util/TokenUtils.java index 64e58d0..d1aa0aa 100644 --- a/screen-common/src/main/java/com/moral/util/TokenUtils.java +++ b/screen-common/src/main/java/com/moral/util/TokenUtils.java @@ -34,6 +34,15 @@ //token��������� ������������ private static final int validity_time = 60*30; + //������user_token���������������������token + public static boolean hHasKey(String uid){ + return redisTemplate.opsForHash().hasKey("user_token",uid); + } + + //������������id������token + public static Object hget(String uid){ + return redisTemplate.opsForHash().get("user_token",uid); + } /** * @Description: ������token,��������������������������������� * @Param: [uid] type��� ���������������manage ���������������api -- Gitblit v1.8.0