From a5ccfcd3f3e8d96250c92638cf2f66736827ffbc Mon Sep 17 00:00:00 2001
From: Vitaly Takmazov
Date: Fri, 9 Dec 2022 14:45:24 +0300
Subject: Better caching in AnythingFilter

---
 src/main/java/com/juick/service/UserService.java        |  7 +++++--
 src/main/java/com/juick/service/UserServiceImpl.java    |  6 ++++++
 src/main/java/com/juick/www/filters/AnythingFilter.java | 11 +++++------
 3 files changed, 16 insertions(+), 8 deletions(-)

(limited to 'src/main/java')

diff --git a/src/main/java/com/juick/service/UserService.java b/src/main/java/com/juick/service/UserService.java
index a3ab8315..db02c34e 100644
--- a/src/main/java/com/juick/service/UserService.java
+++ b/src/main/java/com/juick/service/UserService.java
@@ -46,12 +46,15 @@ public interface UserService {
 
     String getSignUpHashByTelegramID(Long telegramId, String username);
 
-    @CacheEvict(value = "users_by_name", allEntries = true)
+    @CacheEvict(value = { "usernames", "users_by_name" }, allEntries = true)
     Optional<User> createUser(String username, String password) throws UsernameTakenException;
 
     Optional<User> getUserByUID(int uid);
 
-    @Cacheable(value = "users_by_name")
+    @Cacheable(value = "usernames")
+    List<String> getUsernames();
+
+    @Cacheable(value = "users_by_name", key = "{ #username }")
     @Nonnull User getUserByName(String username);
 
     @Nonnull User getUserByEmail(String email);
diff --git a/src/main/java/com/juick/service/UserServiceImpl.java b/src/main/java/com/juick/service/UserServiceImpl.java
index 71f2dc91..0e6f9606 100644
--- a/src/main/java/com/juick/service/UserServiceImpl.java
+++ b/src/main/java/com/juick/service/UserServiceImpl.java
@@ -173,6 +173,12 @@ public class UserServiceImpl extends BaseJdbcService implements UserService {
         return AnonymousUser.INSTANCE;
     }
 
+    @Transactional(readOnly = true)
+    @Override
+    public List<String> getUsernames() {
+        return getJdbcTemplate().queryForList("SELECT nick FROM users", String.class);
+    }
+
     @Override
     @Transactional(readOnly = true)
     @Nonnull
diff --git a/src/main/java/com/juick/www/filters/AnythingFilter.java b/src/main/java/com/juick/www/filters/AnythingFilter.java
index a725a66f..2ec82801 100644
--- a/src/main/java/com/juick/www/filters/AnythingFilter.java
+++ b/src/main/java/com/juick/www/filters/AnythingFilter.java
@@ -71,16 +71,15 @@ public class AnythingFilter extends OncePerRequestFilter {
                     }
                 }
             }
-            User user = userService.getUserByName(anything);
-            if (user.getUid() > 0) {
-                servletResponse.sendRedirect("/" + user.getName() + "/");
+             ;
+            if (userService.getUsernames().stream().anyMatch(name -> name.equals(anything))) {
+                servletResponse.sendRedirect("/" + anything + "/");
             } else {
                 filterChain.doFilter(servletRequest, servletResponse);
             }
         } else {
-            User user = userService.getUserByName(anything);
-            if (!user.isAnonymous()) {
-                servletResponse.sendRedirect("/" + user.getName() + "/?before=" + before);
+            if (userService.getUsernames().stream().anyMatch(name -> name.equals(anything))) {
+                servletResponse.sendRedirect("/" + anything + "/?before=" + before);
             } else {
                 filterChain.doFilter(servletRequest, servletResponse);
             }
-- 
cgit v1.2.3