diff --git a/pjcloud-app-server/pjcloud-app-server-biz/pom.xml b/pjcloud-app-server/pjcloud-app-server-biz/pom.xml index 6788632a8..cfd96feff 100644 --- a/pjcloud-app-server/pjcloud-app-server-biz/pom.xml +++ b/pjcloud-app-server/pjcloud-app-server-biz/pom.xml @@ -13,6 +13,10 @@ pjcloud-app-server-biz + + com.bsd + pjcloud-upms-api + org.springframework.boot diff --git a/pjcloud-common/pjcloud-common-security/src/main/java/com/bsd/pjcloud/common/security/service/PjcloudTocDefaultUserDetailsServiceImpl.java b/pjcloud-common/pjcloud-common-security/src/main/java/com/bsd/pjcloud/common/security/service/PjcloudTocDefaultUserDetailsServiceImpl.java index 30770e223..b93b7fbac 100644 --- a/pjcloud-common/pjcloud-common-security/src/main/java/com/bsd/pjcloud/common/security/service/PjcloudTocDefaultUserDetailsServiceImpl.java +++ b/pjcloud-common/pjcloud-common-security/src/main/java/com/bsd/pjcloud/common/security/service/PjcloudTocDefaultUserDetailsServiceImpl.java @@ -17,6 +17,8 @@ package com.bsd.pjcloud.common.security.service; import cn.hutool.core.util.ArrayUtil; +import com.bsd.pjcloud.admin.api.dto.UserInfo; +import com.bsd.pjcloud.admin.api.feign.RemoteUserService; import com.bsd.pjcloud.app.api.dto.AppUserInfo; import com.bsd.pjcloud.app.api.entity.AppUser; import com.bsd.pjcloud.app.api.feign.RemoteAppUserService; @@ -57,6 +59,8 @@ public class PjcloudTocDefaultUserDetailsServiceImpl implements PjcloudUserDetai private final Optional remoteAppUserServiceOptional; + private final RemoteUserService remoteUserService; + /** * 用户密码登录 * @@ -66,16 +70,26 @@ public class PjcloudTocDefaultUserDetailsServiceImpl implements PjcloudUserDetai @Override @SneakyThrows public UserDetails loadUserByUsername(String username) { - Cache cache = cacheManager.getCache(CacheConstants.USER_DETAILS_MINI); +// Cache cache = cacheManager.getCache(CacheConstants.USER_DETAILS_MINI); +// if (Objects.nonNull(cache) && Objects.nonNull(cache.get(username))) { +// AppUserInfo appUserInfo = cache.get(username, AppUserInfo.class); +// return getUserDetailsAppUser(R.ok(appUserInfo)); +// } +// +// R result = remoteAppUserServiceOptional.get().info(username); +// Optional userInfoOptional = RetOps.of(result).getData(); +// userInfoOptional.ifPresent(userInfo -> Objects.requireNonNull(cache).put(username, userInfo)); +// return getUserDetailsAppUser(result); + + Cache cache = cacheManager.getCache(CacheConstants.USER_DETAILS); if (Objects.nonNull(cache) && Objects.nonNull(cache.get(username))) { - AppUserInfo appUserInfo = cache.get(username, AppUserInfo.class); - return getUserDetailsAppUser(R.ok(appUserInfo)); + return getUserDetails(Optional.ofNullable(cache.get(username, UserInfo.class))); } - R result = remoteAppUserServiceOptional.get().info(username); - Optional userInfoOptional = RetOps.of(result).getData(); + R result = remoteUserService.info(username); + Optional userInfoOptional = RetOps.of(result).getData(); userInfoOptional.ifPresent(userInfo -> Objects.requireNonNull(cache).put(username, userInfo)); - return getUserDetailsAppUser(result); + return getUserDetails(userInfoOptional); } @Override