Fix perf

src/main/java/com/darkfantasy/config/WebConfig.java

@@ -1,11 +1,6 @@
 package com.darkfantasy.config;
 
-import jakarta.servlet.FilterChain;
-import jakarta.servlet.ServletException;
-import jakarta.servlet.http.HttpServletRequest;
-import jakarta.servlet.http.HttpServletResponse;
 import org.springframework.context.annotation.Configuration;
-import org.springframework.web.filter.OncePerRequestFilter;
 import org.springframework.web.servlet.config.annotation.InterceptorRegistry;
 import org.springframework.web.servlet.config.annotation.ResourceHandlerRegistry;
 import org.springframework.web.servlet.config.annotation.WebMvcConfigurer;
@@ -14,8 +9,6 @@ import com.darkfantasy.security.ForceChangePasswordInterceptor;
 
 import lombok.RequiredArgsConstructor;
 
-import java.io.IOException;
-
 @Configuration
 @RequiredArgsConstructor
 public class WebConfig implements WebMvcConfigurer {
@@ -31,20 +24,4 @@ public class WebConfig implements WebMvcConfigurer {
     public void addInterceptors(InterceptorRegistry registry) {
         registry.addInterceptor(interceptor);
     }
-
-    @org.springframework.context.annotation.Bean
-    public OncePerRequestFilter pathTraversalFilter() {
-        return new OncePerRequestFilter() {
-            @Override
-            protected void doFilterInternal(HttpServletRequest request, HttpServletResponse response, FilterChain filterChain)
-                    throws ServletException, IOException {
-                String path = request.getRequestURI();
-                if (path.contains("..") || path.contains("./")) {
-                    response.sendError(HttpServletResponse.SC_FORBIDDEN, "Access Denied");
-                    return;
-                }
-                filterChain.doFilter(request, response);
-            }
-        };
-    }
 }

src/main/java/com/darkfantasy/service/impl/ArticleServiceImpl.java

@@ -2,9 +2,6 @@ package com.darkfantasy.service.impl;
 
 import java.util.List;
 
-import org.springframework.cache.annotation.CacheEvict;
-import org.springframework.cache.annotation.CachePut;
-import org.springframework.cache.annotation.Cacheable;
 import org.springframework.data.domain.Page;
 import org.springframework.data.domain.PageRequest;
 import org.springframework.data.domain.Pageable;
@@ -39,7 +36,6 @@ public class ArticleServiceImpl implements ArticleService {
         return ArticleResponse.fromEntity(findArticle(id));
     }
 
-    @CacheEvict(value = "articles", allEntries = true)
     @Transactional
     @Override
     public ArticleResponse createArticle(CreateArticleRequest request) {
@@ -58,7 +54,6 @@ public class ArticleServiceImpl implements ArticleService {
         return ArticleResponse.fromEntity(savedArticle);
     }
 
-    @CacheEvict(value = "articles", allEntries = true)
     @Transactional
     @Override
     public ArticleResponse updateArticle(UpdateArticleRequest request) {
@@ -90,7 +85,6 @@ public class ArticleServiceImpl implements ArticleService {
         return ArticleResponse.fromEntity(article);
     }
 
-    @CacheEvict(value = "articles", allEntries = true)
     @Transactional
     @Override
     public void deleteArticle(Long articleId) {
@@ -108,14 +102,12 @@ public class ArticleServiceImpl implements ArticleService {
                 "Xóa bài viết: " + article.getTitle());
     }
 
-    @Cacheable(value = "articles", key = "#pageable.pageNumber + '_' + #pageable.pageSize")
     @Override
     public Page<ArticleResponse> getArticlesDeletedFalse(Pageable pageable) {
         return articleRepository.findByDeletedFalseOrderByCreatedAtDesc(pageable)
                 .map(ArticleResponse::fromEntity);
     }
 
-    @Cacheable(value = "articles", key = "'latest_' + #limit")
     @Override
     public List<ArticleResponse> getLatestArticles(int limit) {
 
@@ -127,7 +119,6 @@ public class ArticleServiceImpl implements ArticleService {
                 .toList();
     }
 
-    @Cacheable(value = "articles", key = "'important'")
     @Override
     public ArticleResponse getLatestImportantArticle() {
 
@@ -145,7 +136,6 @@ public class ArticleServiceImpl implements ArticleService {
         return articleRepository.findAll(pageable).map(ArticleResponse::fromEntity);
     }
 
-    @CacheEvict(value = "articles", allEntries = true)
     @Transactional
     @Override
     public void restoreArticle(Long articleId) {

src/main/java/com/darkfantasy/service/impl/ContributorServiceImpl.java

@@ -2,8 +2,6 @@ package com.darkfantasy.service.impl;
 
 import java.util.List;
 
-import org.springframework.cache.annotation.CacheEvict;
-import org.springframework.cache.annotation.Cacheable;
 import org.springframework.data.domain.Page;
 import org.springframework.data.domain.PageRequest;
 import org.springframework.data.domain.Pageable;
@@ -37,7 +35,6 @@ public class ContributorServiceImpl implements ContributorService {
         return ContributorResponse.fromEntity(findContributor(id));
     }
 
-    @CacheEvict(value = "contributors", allEntries = true)
     @Transactional
     @Override
     public ContributorResponse createContributor(CreateContributorRequest request) {
@@ -56,7 +53,6 @@ public class ContributorServiceImpl implements ContributorService {
         return ContributorResponse.fromEntity(savedContributor);
     }
 
-    @CacheEvict(value = "contributors", allEntries = true)
     @Transactional
     @Override
     public ContributorResponse updateContributor(UpdateContributorRequest request) {
@@ -88,7 +84,6 @@ public class ContributorServiceImpl implements ContributorService {
         return ContributorResponse.fromEntity(contributor);
     }
 
-    @CacheEvict(value = "contributors", allEntries = true)
     @Transactional
     @Override
     public void deleteContributor(Long contributorId) {
@@ -106,7 +101,6 @@ public class ContributorServiceImpl implements ContributorService {
                 "Xóa người đóng góp: " + contributor.getName());
     }
 
-    @Cacheable(value = "contributors", key = "#pageable.pageNumber + '_' + #pageable.pageSize")
     @Override
     public Page<ContributorResponse> getContributorsDeletedFalse(Pageable pageable) {
         return contributorRepository.findByDeletedFalseOrderByPriorityDesc(pageable)
@@ -121,7 +115,6 @@ public class ContributorServiceImpl implements ContributorService {
         return contributorRepository.findAll(pageable).map(ContributorResponse::fromEntity);
     }
 
-    @CacheEvict(value = "contributors", allEntries = true)
     @Transactional
     @Override
     public void restoreContributor(Long contributorId) {

src/main/java/com/darkfantasy/service/impl/FaqServiceImpl.java

@@ -2,8 +2,6 @@ package com.darkfantasy.service.impl;
 
 import java.util.List;
 
-import org.springframework.cache.annotation.CacheEvict;
-import org.springframework.cache.annotation.Cacheable;
 import org.springframework.data.domain.Page;
 import org.springframework.data.domain.Pageable;
 import org.springframework.stereotype.Service;
@@ -43,7 +41,6 @@ public class FaqServiceImpl implements FaqService {
         return FaqResponse.fromEntity(findFaq(id));
     }
 
-    @CacheEvict(value = "faqs", allEntries = true)
     @Transactional
     @Override
     public FaqResponse createFaq(CreateFaqRequest request) {
@@ -63,7 +60,6 @@ public class FaqServiceImpl implements FaqService {
         return FaqResponse.fromEntity(faq);
     }
 
-    @CacheEvict(value = "faqs", allEntries = true)
     @Transactional
     @Override
     public FaqResponse updateFaq(UpdateFaqRequest request) {
@@ -84,7 +80,6 @@ public class FaqServiceImpl implements FaqService {
         return FaqResponse.fromEntity(faq);
     }
 
-    @CacheEvict(value = "faqs", allEntries = true)
     @Transactional
     @Override
     public void deleteFaq(Long id) {
@@ -101,7 +96,6 @@ public class FaqServiceImpl implements FaqService {
                 "Xóa FAQ: " + found.getTitle());
     }
 
-    @CacheEvict(value = "faqs", allEntries = true)
     @Transactional
     @Override
     public void restoreFaq(Long id) {
@@ -118,7 +112,6 @@ public class FaqServiceImpl implements FaqService {
                 "Khôi phục FAQ: " + found.getTitle());
     }
 
-    @Cacheable(value = "faqs", key = "'all'")
     @Override
     public List<FaqResponse> getFaqsDeletedFalse() {
         return faqRepository

src/main/java/com/darkfantasy/service/impl/GameCharacterServiceImpl.java

@@ -2,8 +2,6 @@ package com.darkfantasy.service.impl;
 
 import java.util.List;
 
-import org.springframework.cache.annotation.CacheEvict;
-import org.springframework.cache.annotation.Cacheable;
 import org.springframework.data.domain.Page;
 import org.springframework.data.domain.PageRequest;
 import org.springframework.data.domain.Pageable;
@@ -38,7 +36,6 @@ public class GameCharacterServiceImpl implements GameCharacterService {
         return GameCharacterResponse.fromEntity(findGameCharacter(id));
     }
 
-    @CacheEvict(value = "characters", allEntries = true)
     @Transactional
     @Override
     public GameCharacterResponse createGameCharacter(
@@ -62,7 +59,6 @@ public class GameCharacterServiceImpl implements GameCharacterService {
                 savedCharacter);
     }
 
-    @CacheEvict(value = "characters", allEntries = true)
     @Transactional
     @Override
     public GameCharacterResponse updateGameCharacter(UpdateGameCharacterRequest request) {
@@ -89,7 +85,6 @@ public class GameCharacterServiceImpl implements GameCharacterService {
 
     }
 
-    @CacheEvict(value = "characters", allEntries = true)
     @Transactional
     @Override
     public void deleteCharacter(Long id) {
@@ -106,7 +101,6 @@ public class GameCharacterServiceImpl implements GameCharacterService {
                 "Xóa nhân vật: " + found.getName());
     }
 
-    @CacheEvict(value = "characters", allEntries = true)
     @Transactional
     @Override
     public void restoreCharacter(Long id) {
@@ -130,7 +124,6 @@ public class GameCharacterServiceImpl implements GameCharacterService {
                 .map(GameCharacterResponse::fromEntity);
     }
 
-    @Cacheable(value = "characters", key = "'limit_' + #limit")
     @Override
     public List<GameCharacterResponse> getCharactersLimit(int limit) {
         if (limit <= 0)
@@ -141,7 +134,6 @@ public class GameCharacterServiceImpl implements GameCharacterService {
                 .map(GameCharacterResponse::fromEntity).toList();
     }
 
-    @Cacheable(value = "characters", key = "'all'")
     @Override
     public List<GameCharacterResponse> getCharactersDeletedFalse() {
         return gameCharacterRepository

src/main/java/com/darkfantasy/service/impl/StoryServiceImpl.java

@@ -2,8 +2,6 @@ package com.darkfantasy.service.impl;
 
 import java.util.List;
 
-import org.springframework.cache.annotation.CacheEvict;
-import org.springframework.cache.annotation.Cacheable;
 import org.springframework.data.domain.Page;
 import org.springframework.data.domain.Pageable;
 import org.springframework.stereotype.Service;
@@ -43,7 +41,6 @@ public class StoryServiceImpl implements StoryService {
         return StoryResponse.fromEntity(findStory(id));
     }
 
-    @CacheEvict(value = "stories", allEntries = true)
     @Transactional
     @Override
     public StoryResponse createStory(CreateStoryRequest request) {
@@ -65,7 +62,6 @@ public class StoryServiceImpl implements StoryService {
         return StoryResponse.fromEntity(savedStory);
     }
 
-    @CacheEvict(value = "stories", allEntries = true)
     @Transactional
     @Override
     public StoryResponse updateStory(UpdateStoryRequest request) {
@@ -92,7 +88,6 @@ public class StoryServiceImpl implements StoryService {
         return StoryResponse.fromEntity(story);
     }
 
-    @CacheEvict(value = "stories", allEntries = true)
     @Transactional
     @Override
     public void deleteStory(Long id) {
@@ -109,7 +104,6 @@ public class StoryServiceImpl implements StoryService {
                 "Xóa câu chuyện: " + found.getTitle());
     }
 
-    @CacheEvict(value = "stories", allEntries = true)
     @Transactional
     @Override
     public void restoreStory(Long id) {
@@ -126,7 +120,6 @@ public class StoryServiceImpl implements StoryService {
                 "Khôi phục câu chuyện: " + found.getTitle());
     }
 
-    @Cacheable(value = "stories", key = "'highest_priority'")
     @Override
     public StoryResponse getStoryDeletedFalseHighestPriority() {
         Story story = storyRepository.findTopByDeletedFalseOrderByPriorityDesc();
@@ -136,7 +129,6 @@ public class StoryServiceImpl implements StoryService {
         return StoryResponse.fromEntity(story);
     }
 
-    @Cacheable(value = "stories", key = "'all'")
     @Override
     public List<StoryResponse> getStoriesDeletedFalse() {
         return storyRepository

src/main/java/com/darkfantasy/service/impl/WorldServiceImpl.java

@@ -2,8 +2,6 @@ package com.darkfantasy.service.impl;
 
 import java.util.List;
 
-import org.springframework.cache.annotation.CacheEvict;
-import org.springframework.cache.annotation.Cacheable;
 import org.springframework.data.domain.Page;
 import org.springframework.data.domain.Pageable;
 import org.springframework.stereotype.Service;
@@ -44,7 +42,6 @@ public class WorldServiceImpl implements WorldService {
         return WorldResponse.fromEntity(findWorld(id));
     }
 
-    @CacheEvict(value = "worlds", allEntries = true)
     @Transactional
     @Override
     public WorldResponse createWorld(CreateWorldRequest request) {
@@ -64,7 +61,6 @@ public class WorldServiceImpl implements WorldService {
         return WorldResponse.fromEntity(world);
     }
 
-    @CacheEvict(value = "worlds", allEntries = true)
     @Transactional
     @Override
     public WorldResponse updateWorld(UpdateWorldRequest request) {
@@ -89,7 +85,6 @@ public class WorldServiceImpl implements WorldService {
         return WorldResponse.fromEntity(world);
     }
 
-    @CacheEvict(value = "worlds", allEntries = true)
     @Transactional
     @Override
     public void deleteWorld(Long id) {
@@ -106,7 +101,6 @@ public class WorldServiceImpl implements WorldService {
                 "Xóa thế giới: " + found.getTitle());
     }
 
-    @CacheEvict(value = "worlds", allEntries = true)
     @Transactional
     @Override
     public void restoreWorld(Long id) {
@@ -123,7 +117,6 @@ public class WorldServiceImpl implements WorldService {
                 "Khôi phục thế giới: " + found.getTitle());
     }
 
-    @Cacheable(value = "worlds", key = "'highest_priority'")
     @Override
     public WorldResponse getWorldDeletedFalseHighestPriority() {
         World world = worldRepository
@@ -136,7 +129,6 @@ public class WorldServiceImpl implements WorldService {
         return WorldResponse.fromEntity(world);
     }
 
-    @Cacheable(value = "worlds", key = "'all'")
     @Override
     public List<WorldResponse> getWorldsDeletedFalse() {
         return worldRepository

src/main/resources/application.properties

@@ -49,6 +49,7 @@ server.error.include-stacktrace=never
 server.error.include-exception=false
 
 # Cache Configuration
-spring.cache.type=caffeine
-spring.cache.caffeine.spec=maximumSize=500,expireAfterWrite=5m
-spring.cache.cache-names=articles,characters,worlds,stories,faqs,contributors
+spring.cache.type=none
+# spring.cache.type=caffeine
+# spring.cache.caffeine.spec=maximumSize=500,expireAfterWrite=5m
+# spring.cache.cache-names=articles,characters,worlds,stories,faqs,contributors