Merge pull request #139 from LionnoiL/118-add-cahing-to-user-findbyid

adding caching to findById for user

pom.xml

@@ -164,6 +164,12 @@
             <version>6.1.7.Final</version>
         </dependency>
 
+        <dependency>
+            <groupId>org.springframework.boot</groupId>
+            <artifactId>spring-boot-starter-cache</artifactId>
+            <version>3.3.0</version>
+        </dependency>
+
         <dependency>
             <groupId>org.springframework.boot</groupId>
             <artifactId>spring-boot-starter-test</artifactId>

src/main/java/org/petmarket/config/CachingConfig.java

@@ -0,0 +1,16 @@
+package org.petmarket.config;
+
+import org.springframework.cache.CacheManager;
+import org.springframework.cache.annotation.EnableCaching;
+import org.springframework.cache.concurrent.ConcurrentMapCacheManager;
+import org.springframework.context.annotation.Bean;
+import org.springframework.context.annotation.Configuration;
+
+@Configuration
+@EnableCaching
+public class CachingConfig {
+    @Bean
+    public CacheManager cacheManager() {
+        return new ConcurrentMapCacheManager("users");
+    }
+}

src/main/java/org/petmarket/users/service/UserService.java

@@ -28,6 +28,7 @@
 import org.petmarket.users.repository.UserRepository;
 import org.springframework.beans.BeanUtils;
 import org.springframework.beans.factory.annotation.Value;
+import org.springframework.cache.annotation.Cacheable;
 import org.springframework.security.core.Authentication;
 import org.springframework.security.core.context.SecurityContextHolder;
 import org.springframework.stereotype.Service;
@@ -86,6 +87,7 @@ public boolean isCurrentUserAdmin() {
         return auth != null && auth.getAuthorities().stream().anyMatch(a -> a.getAuthority().equals("ROLE_ADMIN"));
     }
 
+    @Cacheable(value = "users", key = "#userId")
     public User findById(Long userId) {
         return userRepository.findById(userId)
                 .orElseThrow(() -> new ItemNotFoundException("User not found by id: " + userId));