提交 81930d53 编写于 作者: O o2sword

个人忘记密码安全性修改

上级 4effc19b
...@@ -5,12 +5,19 @@ import com.x.base.core.container.factory.EntityManagerContainerFactory; ...@@ -5,12 +5,19 @@ import com.x.base.core.container.factory.EntityManagerContainerFactory;
import com.x.base.core.project.config.Config; import com.x.base.core.project.config.Config;
import com.x.base.core.project.http.ActionResult; import com.x.base.core.project.http.ActionResult;
import com.x.base.core.project.http.WrapOutBoolean; import com.x.base.core.project.http.WrapOutBoolean;
import com.x.base.core.project.logger.Logger;
import com.x.base.core.project.logger.LoggerFactory;
import com.x.base.core.project.tools.Crypto;
import com.x.base.core.project.tools.DefaultCharset;
import com.x.organization.assemble.personal.Business; import com.x.organization.assemble.personal.Business;
import com.x.organization.core.entity.Person; import com.x.organization.core.entity.Person;
import org.apache.commons.lang3.BooleanUtils; import org.apache.commons.lang3.BooleanUtils;
import org.codehaus.plexus.util.StringUtils;
class ActionCode extends BaseAction { import java.net.URLDecoder;
class ActionCode extends BaseAction {
private static final Logger LOGGER = LoggerFactory.getLogger(ActionCode.class);
ActionResult<WrapOutBoolean> execute(String credential) throws Exception { ActionResult<WrapOutBoolean> execute(String credential) throws Exception {
try (EntityManagerContainer emc = EntityManagerContainerFactory.instance().create()) { try (EntityManagerContainer emc = EntityManagerContainerFactory.instance().create()) {
ActionResult<WrapOutBoolean> result = new ActionResult<>(); ActionResult<WrapOutBoolean> result = new ActionResult<>();
...@@ -19,6 +26,9 @@ class ActionCode extends BaseAction { ...@@ -19,6 +26,9 @@ class ActionCode extends BaseAction {
if (BooleanUtils.isNotTrue(Config.collect().getEnable())) { if (BooleanUtils.isNotTrue(Config.collect().getEnable())) {
throw new ExceptionDisableCollect(); throw new ExceptionDisableCollect();
} }
credential = BooleanUtils.isTrue(Config.token().getRsaEnable()) ? Crypto.rsaDecrypt(URLDecoder.decode(credential, DefaultCharset.charset),
Config.privateKey()) : credential;
LOGGER.info("{} 用户进行忘记密码修改操作", credential);
Person person = business.person().getWithCredential(credential); Person person = business.person().getWithCredential(credential);
if (null == person) { if (null == person) {
throw new ExceptionSendCodeError(); throw new ExceptionSendCodeError();
......
...@@ -44,9 +44,11 @@ class ActionReset extends BaseAction { ...@@ -44,9 +44,11 @@ class ActionReset extends BaseAction {
if (StringUtils.isBlank(password)) { if (StringUtils.isBlank(password)) {
throw new ExceptionPasswordEmpty(); throw new ExceptionPasswordEmpty();
} }
credential = BooleanUtils.isTrue(Config.token().getRsaEnable()) ? Crypto.rsaDecrypt(credential, Config.privateKey()) : credential;
password = BooleanUtils.isTrue(Config.token().getRsaEnable()) ? Crypto.rsaDecrypt(password, Config.privateKey()) : password;
Person person = business.person().getWithCredential(credential); Person person = business.person().getWithCredential(credential);
if (null == person) { if (null == person) {
throw new ExceptionPersonNotExist(credential); throw new ExceptionPersonNotExistOrInvalidAnswer();
} }
person = emc.find(person.getId(), Person.class, ExceptionWhen.not_found); person = emc.find(person.getId(), Person.class, ExceptionWhen.not_found);
if (BooleanUtils.isTrue(Config.person().getSuperPermission()) if (BooleanUtils.isTrue(Config.person().getSuperPermission())
...@@ -57,7 +59,7 @@ class ActionReset extends BaseAction { ...@@ -57,7 +59,7 @@ class ActionReset extends BaseAction {
throw new ExceptionInvalidPassword(Config.person().getPasswordRegexHint()); throw new ExceptionInvalidPassword(Config.person().getPasswordRegexHint());
} }
if (BooleanUtils.isFalse(business.instrument().code().validate(person.getMobile(), codeAnswer))) { if (BooleanUtils.isFalse(business.instrument().code().validate(person.getMobile(), codeAnswer))) {
throw new ExceptionInvalidCode(); throw new ExceptionPersonNotExistOrInvalidAnswer();
} }
} }
emc.beginTransaction(Person.class); emc.beginTransaction(Person.class);
......
package com.x.organization.assemble.personal.jaxrs.reset; package com.x.organization.assemble.personal.jaxrs.reset;
import org.apache.commons.codec.binary.Base64;
import org.apache.commons.lang3.BooleanUtils;
import org.apache.commons.lang3.StringUtils;
import com.google.gson.JsonElement; import com.google.gson.JsonElement;
import com.x.base.core.container.EntityManagerContainer; import com.x.base.core.container.EntityManagerContainer;
import com.x.base.core.container.factory.EntityManagerContainerFactory; import com.x.base.core.container.factory.EntityManagerContainerFactory;
import com.x.base.core.project.annotation.FieldDescribe; import com.x.base.core.project.annotation.FieldDescribe;
import com.x.base.core.project.cache.CacheManager; import com.x.base.core.project.cache.CacheManager;
import com.x.base.core.project.config.Config; import com.x.base.core.project.config.Config;
import com.x.base.core.project.exception.ExceptionPersonNotExist;
import com.x.base.core.project.gson.GsonPropertyObject; import com.x.base.core.project.gson.GsonPropertyObject;
import com.x.base.core.project.http.ActionResult; import com.x.base.core.project.http.ActionResult;
import com.x.base.core.project.http.EffectivePerson; import com.x.base.core.project.http.EffectivePerson;
...@@ -20,6 +15,9 @@ import com.x.base.core.project.logger.LoggerFactory; ...@@ -20,6 +15,9 @@ import com.x.base.core.project.logger.LoggerFactory;
import com.x.base.core.project.tools.Crypto; import com.x.base.core.project.tools.Crypto;
import com.x.organization.assemble.personal.Business; import com.x.organization.assemble.personal.Business;
import com.x.organization.core.entity.Person; import com.x.organization.core.entity.Person;
import org.apache.commons.codec.binary.Base64;
import org.apache.commons.lang3.BooleanUtils;
import org.apache.commons.lang3.StringUtils;
public class ActionSetPasswordAnonymous extends BaseAction { public class ActionSetPasswordAnonymous extends BaseAction {
private static final Logger LOGGER = LoggerFactory.getLogger(ActionSetPasswordAnonymous.class); private static final Logger LOGGER = LoggerFactory.getLogger(ActionSetPasswordAnonymous.class);
...@@ -30,7 +28,6 @@ public class ActionSetPasswordAnonymous extends BaseAction { ...@@ -30,7 +28,6 @@ public class ActionSetPasswordAnonymous extends BaseAction {
Wi wi = this.convertToWrapIn(jsonElement, Wi.class); Wi wi = this.convertToWrapIn(jsonElement, Wi.class);
Business business = new Business(emc); Business business = new Business(emc);
/** 排除xadmin */
if (Config.token().isInitialManager(wi.getUserName())) { if (Config.token().isInitialManager(wi.getUserName())) {
throw new ExceptionEditInitialManagerDeny(); throw new ExceptionEditInitialManagerDeny();
} else { } else {
...@@ -40,13 +37,10 @@ public class ActionSetPasswordAnonymous extends BaseAction { ...@@ -40,13 +37,10 @@ public class ActionSetPasswordAnonymous extends BaseAction {
Person o = business.person().getWithCredential(wi.getUserName()); Person o = business.person().getWithCredential(wi.getUserName());
if (null == o) { if (null == o) {
throw new ExceptionPersonNotExist(wi.getUserName()); throw new ExceptionPersonNotExistOrInvalidPassword();
} }
Person person = emc.find(o.getId(), Person.class); Person person = emc.find(o.getId(), Person.class);
if (null == person) {
throw new ExceptionPersonNotExist(wi.getUserName());
}
if (StringUtils.isEmpty(wi.getOldPassword())) { if (StringUtils.isEmpty(wi.getOldPassword())) {
throw new ExceptionOldPasswordEmpty(); throw new ExceptionOldPasswordEmpty();
...@@ -54,31 +48,23 @@ public class ActionSetPasswordAnonymous extends BaseAction { ...@@ -54,31 +48,23 @@ public class ActionSetPasswordAnonymous extends BaseAction {
if (StringUtils.isEmpty(wi.getNewPassword())) { if (StringUtils.isEmpty(wi.getNewPassword())) {
throw new ExceptionPasswordEmpty(); throw new ExceptionPasswordEmpty();
} }
if (StringUtils.isEmpty(wi.getConfirmPassword())) { if (StringUtils.isEmpty(wi.getConfirmPassword())) {
throw new ExceptionConfirmPasswordEmpty(); throw new ExceptionConfirmPasswordEmpty();
} }
if (!StringUtils.equals(wi.getNewPassword(), wi.getConfirmPassword())) { String oldPassword = BooleanUtils.isTrue(Config.token().getRsaEnable()) ? Crypto.rsaDecrypt(wi.getOldPassword(), Config.privateKey())
throw new ExceptionTwicePasswordNotMatch(); : wi.getOldPassword();
} String newPassword = BooleanUtils.isTrue(Config.token().getRsaEnable()) ? Crypto.rsaDecrypt(wi.getNewPassword(), Config.privateKey())
: wi.getNewPassword();
String confirmPassword = BooleanUtils.isTrue(Config.token().getRsaEnable()) ? Crypto.rsaDecrypt(wi.getConfirmPassword(), Config.privateKey())
: wi.getConfirmPassword();
if (StringUtils.equals(wi.getNewPassword(), wi.getOldPassword())) { if (StringUtils.equals(wi.getNewPassword(), wi.getOldPassword())) {
throw new ExceptionNewPasswordSameAsOldPassword(); throw new ExceptionNewPasswordSameAsOldPassword();
} }
String oldPassword = wi.getOldPassword(); if(!StringUtils.equals(newPassword, confirmPassword)){
String newPassword = wi.getNewPassword(); throw new ExceptionTwicePasswordNotMatch();
String confirmPassword = wi.getConfirmPassword();
String isEncrypted = wi.getIsEncrypted();
// RSA解秘
if (!StringUtils.isEmpty(isEncrypted)) {
if (isEncrypted.trim().equalsIgnoreCase("y")) {
oldPassword = this.decryptRSA(oldPassword);
newPassword = this.decryptRSA(newPassword);
confirmPassword = this.decryptRSA(confirmPassword);
}
} }
if (BooleanUtils.isTrue(Config.person().getSuperPermission()) if (BooleanUtils.isTrue(Config.person().getSuperPermission())
...@@ -88,7 +74,7 @@ public class ActionSetPasswordAnonymous extends BaseAction { ...@@ -88,7 +74,7 @@ public class ActionSetPasswordAnonymous extends BaseAction {
if (!StringUtils.equals( if (!StringUtils.equals(
Crypto.encrypt(oldPassword, Config.token().getKey(), Config.person().getEncryptType()), Crypto.encrypt(oldPassword, Config.token().getKey(), Config.person().getEncryptType()),
person.getPassword())) { person.getPassword())) {
throw new ExceptionOldPasswordNotMatch(); throw new ExceptionPersonNotExistOrInvalidPassword();
} }
if (!newPassword.matches(Config.person().getPasswordRegex())) { if (!newPassword.matches(Config.person().getPasswordRegex())) {
throw new ExceptionInvalidPassword(Config.person().getPasswordRegexHint()); throw new ExceptionInvalidPassword(Config.person().getPasswordRegexHint());
......
package com.x.organization.assemble.personal.jaxrs.reset;
import com.x.base.core.project.exception.PromptException;
class ExceptionPersonNotExistOrInvalidAnswer extends PromptException {
private static final long serialVersionUID = -8334021007462970656L;
public static String defaultMessage = "用户不存在或者验证码错误.";
ExceptionPersonNotExistOrInvalidAnswer( ) {
super(defaultMessage);
}
}
package com.x.organization.assemble.personal.jaxrs.reset;
import com.x.base.core.project.exception.PromptException;
class ExceptionPersonNotExistOrInvalidPassword extends PromptException {
private static final long serialVersionUID = 2537120821114609351L;
public static String defaultMessage = "用户不存在或者密码错误.";
ExceptionPersonNotExistOrInvalidPassword( ) {
super(defaultMessage);
}
}
package com.x.organization.assemble.personal.jaxrs.reset; package com.x.organization.assemble.personal.jaxrs.reset;
import javax.servlet.http.HttpServletRequest;
import javax.ws.rs.Consumes;
import javax.ws.rs.GET;
import javax.ws.rs.POST;
import javax.ws.rs.PUT;
import javax.ws.rs.Path;
import javax.ws.rs.PathParam;
import javax.ws.rs.Produces;
import javax.ws.rs.container.AsyncResponse;
import javax.ws.rs.container.Suspended;
import javax.ws.rs.core.Context;
import javax.ws.rs.core.MediaType;
import com.google.gson.JsonElement; import com.google.gson.JsonElement;
import com.x.base.core.project.annotation.JaxrsDescribe; import com.x.base.core.project.annotation.JaxrsDescribe;
import com.x.base.core.project.annotation.JaxrsMethodDescribe; import com.x.base.core.project.annotation.JaxrsMethodDescribe;
...@@ -26,6 +13,13 @@ import com.x.base.core.project.jaxrs.StandardJaxrsAction; ...@@ -26,6 +13,13 @@ import com.x.base.core.project.jaxrs.StandardJaxrsAction;
import com.x.base.core.project.logger.Logger; import com.x.base.core.project.logger.Logger;
import com.x.base.core.project.logger.LoggerFactory; import com.x.base.core.project.logger.LoggerFactory;
import javax.servlet.http.HttpServletRequest;
import javax.ws.rs.*;
import javax.ws.rs.container.AsyncResponse;
import javax.ws.rs.container.Suspended;
import javax.ws.rs.core.Context;
import javax.ws.rs.core.MediaType;
@Path("reset") @Path("reset")
@JaxrsDescribe("重置操作") @JaxrsDescribe("重置操作")
public class ResetAction extends StandardJaxrsAction { public class ResetAction extends StandardJaxrsAction {
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册