리프레시 토큰 관리를 Redis로 변경

[cookienc]

관련이슈

• resolved 리프레시 토큰 관리를 Redis로 변경 #704

참고사항

• 기존 RefreshToken로직을 Redis로 변경했습니다.
  • 이 과정에서 Token 관련 기능을 Facade 패턴으로 변경했습니다 -> Oauth는 토큰 관리보다는� login과 같은 회원 인증이 먼저라고 생각하는데 토큰 관련 코드가 절반 이상이라 복잡해 보여서 그랬어용
• docker compose 오류가 있어 Spring Boot 버전을 3.1.1 -> 3.1.8버전으로 변경했습니다.
• CRUD repository를 사용하면 jpa처럼 쓸 수 있어서 redistemplate은 사용 안했습니다.
• 테스트에서만 expire가 필요해서 redistemplate을 빈으로 등록해놨습니다.
• TestContainer를 사용해서 Redis를 띄워서 테스트합니다. m1에서 이슈가 있어요

중요한 내용은 없지만 블로그 글 첨부합니다~�

• Redis를 활용해서 RefreshToken 최적화하기
• Spring에서 Redis Test 하기

[hyena0608]

운영 코드에서는 RedisTemplate을 이용하지 않았는데 테스트에서는 이용하게 된 이유가 궁금합니다!

[hyena0608]

오 블로그 글 봤는데 TypeReference로 해결이 되나 궁금하네요🤔🤔

[cookienc]

1. CrudRepository를 상속하면 Jpa처럼 Redis를 사용할 수 있다는게 장점이라 느껴져서 Redistemplate을 사용하지 않았습니다! 단순히 저장하고 조회하는 기능만 리프레시 토큰에서 사용해서요. 그런데 테스트해서는 토큰 만료를 테스트해야되서 redistemplate을 사용해서 expire 메서드가 필요했습니다. 그래서 테스트 코드에서는 사용하게 되었습니다.

TypeReference를 말씀하신거보니 Jackson2JsonRedisSerializer 같은데요. 이 부분은 설명이 살짝 부족했네요

        redisTemplate.setValueSerializer(new Jackson2JsonRedisSerializer<>(RefreshToken.class));
        redisTemplate.setHashValueSerializer(new Jackson2JsonRedisSerializer<>(RefreshToken.class));

이 Serializer는 다음과 같이 빈으로 등록할 때 타입을 지정해서 등록하기 때문에 TypeReference만으로도 잘 동작합니다!

[hyena0608]

'@DynamicPropertySource'를 사용해보는거는 어떤가요?

[cookienc]

5. An Alternative: Test Fixtures
So far, in both approaches, the fixture setup and the test code are tightly intertwined. Sometimes, this tight coupling of two concerns complicates the test code, especially when we have multiple things to set up. Imagine what the infrastructure setup would look like if we were using PostgreSQL and Apache Kafka in a single test.

In addition to that, the infrastructure setup and applying dynamic configurations will be duplicated in all tests that need them.

밸덩에서 퍼온건데 @DynamicPropertySource는 각각의 테스트에서만 사용할 수 있는듯?
혹시 어떻게 하는지 아시나용?

[hyena0608]

집가서 다시 커멘트 남길게용

[hyena0608]

요런 느낌입니다!
(*자바로는 실험 못해봄 🙃)

public class RedisTestProperties {

    @DynamicPropertySource
    public static void redisTestProperties(DynamicPropertyRegistry registry) {
        registry.add("spring.redis.host", redisContainer.getHost());
        registry.add("spring.redis.port", Integer.toString(redisContainer.getMappedPort(6379).toString());
    }
}

[cookienc]

오호 이거 해봤는데 안되더라구... 레디스테스트컨테이너는 setProperty로 설정을 덮어(?)씌워야되는데(왜인지 모름), 저건 static으로 올리니까 안되는거 같음