Skip to content

Commit cbe3193

Browse files
authored
Merge pull request #95 from PSMRI/fix-redis-check
fix: save minimal data to users_<id> key of Redis
2 parents 362e39e + b3fd98d commit cbe3193

1 file changed

Lines changed: 10 additions & 3 deletions

File tree

src/main/java/com/iemr/ecd/utils/mapper/JwtAuthenticationUtil.java

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -104,6 +104,7 @@ private Users getUserFromCache(String userId) {
104104
return user; // Returns null if not found
105105
}
106106

107+
107108
private Users fetchUserFromDB(String userId) {
108109
// This method will only be called if the user is not found in Redis.
109110
String redisKey = "user_" + userId; // Redis key format
@@ -112,15 +113,21 @@ private Users fetchUserFromDB(String userId) {
112113
Users user = userLoginRepo.getUserByUserID(Long.parseLong(userId));
113114

114115
if (user != null) {
115-
// Cache the user in Redis for future requests (cache for 30 minutes)
116-
redisTemplate.opsForValue().set(redisKey, user, 30, TimeUnit.MINUTES);
116+
Users userHash = new Users();
117+
userHash.setUserID(user.getUserID());
118+
userHash.setUserName(user.getUserName());
119+
120+
// Cache the minimal user in Redis for future requests (cache for 30 minutes)
121+
redisTemplate.opsForValue().set(redisKey, userHash, 30, TimeUnit.MINUTES);
117122

118123
// Log that the user has been stored in Redis
119124
logger.info("User stored in Redis with key: " + redisKey);
125+
126+
return user;
120127
} else {
121128
logger.warn("User not found for userId: " + userId);
122129
}
123130

124-
return user;
131+
return null;
125132
}
126133
}

0 commit comments

Comments
 (0)