Merge pull request #39 from gooiman/feat/logout

fix: blacklist ttl 추가
gooiman · Sep 28, 2024 · bf58f51 · bf58f51
2 parents 8597db2 + 9d3bb5b
commit bf58f51
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 6 deletions.
diff --git a/src/main/java/dev/gooiman/server/auth/application/BlackListService.java b/src/main/java/dev/gooiman/server/auth/application/BlackListService.java
@@ -5,12 +5,16 @@
 import dev.gooiman.server.common.dto.CommonSuccessDto;
 import java.util.Optional;
 import lombok.RequiredArgsConstructor;
+import org.springframework.beans.factory.annotation.Value;
 import org.springframework.stereotype.Service;
 
 @Service
 @RequiredArgsConstructor
 public class BlackListService {
 
+    @Value("${spring.security.blacklist-validity-time}")
+    private Long expiration;
+
     private final BlackListRepository blackListRepository;
 
     public boolean isExists(String id) {
@@ -20,7 +24,7 @@ public boolean isExists(String id) {
     }
 
     public CommonSuccessDto saveBlackList(String token) {
-        BlackList entity = new BlackList(token);
+        BlackList entity = new BlackList(token, expiration);
         blackListRepository.save(entity);
         return CommonSuccessDto.fromEntity(true);
     }

diff --git a/src/main/java/dev/gooiman/server/auth/repository/entity/BlackList.java b/src/main/java/dev/gooiman/server/auth/repository/entity/BlackList.java
@@ -1,7 +1,6 @@
 package dev.gooiman.server.auth.repository.entity;
 
 import lombok.Getter;
-import org.springframework.beans.factory.annotation.Value;
 import org.springframework.data.annotation.Id;
 import org.springframework.data.redis.core.RedisHash;
 import org.springframework.data.redis.core.TimeToLive;
@@ -11,13 +10,13 @@
 public class BlackList {
 
     @Id
-    private String token;
+    private final String token;
 
     @TimeToLive
-    @Value("${spring.security.blacklist-validity-time}")
-    private Long expiration;
+    private final Long expiration;
 
-    public BlackList(String token) {
+    public BlackList(String token, Long expiration) {
         this.token = token;
+        this.expiration = expiration;
     }
 }