[Refactor] WebFlux 기반 필터 전환

[HyeonBin2379]

작업 배경

• 게이트웨이 부하테스트 수행 결과에 기반한 성능 개선 작업

작업 내용

• 기존 서블릿 기반 JwtGatewayFilter의 내부 로직을 WebFlux 기반 필터로 교체
• AuthProvider의 토큰 블랙리스트 포함 여부 검증 로직을 FeignClient에서 WebClient 기반으로 교체
• ConcurrentHashMap을 사용한 로컬 캐시를 Caffeine Cache로 대체
  -> TTL이 만료된 데이터는 자동 삭제

테스트 여부

• gradle clean build 명령어 실행 시 정상적으로 빌드됨을 확인

• JwtGatewayIntegrationTest의 통합테스트 통과 확인

• 현재 PR에서 재배포 후 게이트웨이의 기능 동작 확인 예정
  (제대로 동작하지 않을 경우 configs 파일의 게이트웨이 관련 설정을 이전으로 원상복구 후 dev 브랜치의 코드를 재배포)

WebFlux 필터 전환 및 재배포 후 라우팅 기능 정상 동작 확인

기타

이슈

이 PR과 연관된 이슈 번호를 작성해주세요. (이슈 없으면 생략 가능)

• This closes [FEAT] 게이트웨이 리팩토링 및 README 업데이트 #27

[coderabbitai]

📝 Walkthrough

개요

게이트웨이 인증 파이프라인을 FeignClient 기반 동기식 서블릿 필터에서 WebClient 기반 비동기식 WebFlux 필터로 마이그레이션합니다. Caffeine 캐싱을 도입하고, 보안 설정과 테스트를 모두 리액티브 아키텍처로 재구성합니다.

변경사항

서블릿에서 WebFlux로의 게이트웨이 마이그레이션

Layer / File(s)	요약
의존성 및 기초 설정 변경 build.gradle, docker-compose.yaml, src/main/java/org/pgsg/gateway/GatewayApplication.java	gateway-server-webflux 의존성을 도입하고 Caffeine 캐싱, WireMock 테스트 라이브러리를 추가합니다. docker-compose의 정적 환경 설정을 제거하고, GatewayApplication에서 OpenFeign 활성화를 제거합니다.
AuthProvider 리액티브 변환 src/main/java/org/pgsg/gateway/auth/AuthProvider.java, src/main/java/org/pgsg/gateway/auth/AuthProviderImpl.java	AuthProvider 인터페이스를 boolean 동기식 반환에서 Mono<Boolean> 비동기식으로 변경하고, AuthProviderImpl에서 ConcurrentHashMap 기반 수동 캐싱을 Caffeine 캐시로 교체합니다.
인증 클라이언트 WebClient 도입 src/main/java/org/pgsg/gateway/client/AuthClient.java, src/main/java/org/pgsg/gateway/feign/AuthClient.java (삭제)	WebClient 기반 AuthClient 컴포넌트를 추가하여 user-service의 토큰 검증 엔드포인트를 호출하고, 기존 FeignClient 인터페이스와 Fallback 팩토리를 제거합니다.
Gateway 보안 설정 WebFlux 마이그레이션 src/main/java/org/pgsg/gateway/config/GatewaySecurityConfig.java	GatewaySecurityConfig를 Spring MVC HttpSecurity에서 WebFlux ServerHttpSecurity로 마이그레이션하고, 보안 필터 체인을 WebFlux 기반으로 재구성합니다.
Gateway 애플리케이션 컨텍스트 리팩토링 src/main/java/org/pgsg/gateway/config/GatewayAppCtx.java	GatewayAppCtx에서 서블릿 기반 필터(LoginFilter, SecurityConfig, 예외 핸들러)를 제거하고, 로드밸런싱 기능이 포함된 리액티브 WebClient.Builder 빈을 추가합니다.
JWT 필터 WebFlux 리액티브 전환 src/main/java/org/pgsg/gateway/filter/JwtGatewayFilter.java, src/main/java/org/pgsg/gateway/filter/HttpRequestHeaderWrapper.java (삭제)	JwtGatewayFilter를 OncePerRequestFilter 서블릿 필터에서 GlobalFilter 리액티브 필터로 전환하고, 동기식 토큰 검증·헤더 주입 로직을 Mono 기반 비동기 체인으로 재구성합니다. HttpRequestHeaderWrapper 클래스를 제거하고 ServerHttpRequest.mutate()로 헤더 관리를 통합합니다.
통합 테스트 WebFlux 및 WireMock 전환 src/test/java/org/pgsg/gateway/JwtGatewayIntegrationTest.java	JwtGatewayIntegrationTest를 MockMvc 기반 로컬 컨트롤러에서 WebTestClient 기반 WireMock 다운스트림 검증으로 전환하고, AuthProvider 모킹을 Mono 리액티브 반환 방식으로 업데이트합니다.

추정 코드 리뷰 투입도

🎯 4 (Complex) | ⏱️ ~45분

관련 PR

• 89-49/gateway-server#7: JwtGatewayFilter를 WebFlux 기반 필터로 확장하고 JwtGatewayIntegrationTest를 WireMock 기반으로 전환하는 직접적인 후속 작업입니다.
• 89-49/gateway-server#24: 같은 JwtGatewayFilter의 WHITELIST 경로 설정을 수정하므로 코드 범위가 겹칩니다.
• 89-49/gateway-server#5: AuthProvider/AuthProviderImpl의 토큰 검증·블랙리스트 흐름을 서블릿에서 WebFlux로 전환하는 직접적인 변경입니다.

추천 리뷰어

• ddangme
• loveletheart
• Jin4041

축하 시

🐰 ✨ 아 정말 신기한데요,
리액티브한 필터로 깔끔히 변신하고,
웹플럭스의 모나드 시를 부르네요! 📡
카페인 캐시가 속도를 높여주고,
와이어목 테스트로 신뢰도 쌓아올려요! 🚀

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	PR은 이슈 #27의 모든 주요 코딩 요구사항을 충족합니다: JwtGatewayFilter를 WebFlux 기반으로 전환, FeignClient를 WebClient로 대체, 보안 설정을 WebFlux로 수정했습니다.
Out of Scope Changes check	✅ Passed	build.gradle의 의존성 변경, docker-compose 환경변수 제거 등은 모두 WebFlux 전환을 위한 필수 변경이며 요구사항 범위 내입니다.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch refactor/#27-gateway-filter-refactor

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

🧹 Nitpick comments (3)

src/main/java/org/pgsg/gateway/client/AuthClient.java (2)

10-10: 💤 Low value

주석 처리된 코드 제거 권장.

FeignClient 관련 주석 처리된 코드가 남아 있습니다. 이 PR에서 WebClient로 완전히 전환되었으므로 해당 주석을 제거하는 것이 좋습니다.

🧹 주석 제거

-//@FeignClient(name = "user-service", fallbackFactory = AuthClientFallbackFactory.class)
 `@Component`
 public class AuthClient {

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@src/main/java/org/pgsg/gateway/client/AuthClient.java` at line 10, Remove the
leftover commented FeignClient annotation line in AuthClient.java (the line
containing //@FeignClient(name = "user-service", fallbackFactory =
AuthClientFallbackFactory.class)) since the project has switched to WebClient;
simply delete that commented line so the codebase no longer contains unused
Feign-related comments.

---

20-27: ⚡ Quick win

외부 서비스 호출에 타임아웃 설정 권장.

user-service 호출 시 타임아웃이 설정되어 있지 않습니다. 외부 서비스 장애 시 요청이 무한정 대기할 수 있어 게이트웨이 안정성에 영향을 줄 수 있습니다.

⏱️ 타임아웃 설정 예시

 public Mono<CommonResponse<AuthDto.TokenVerifyData>> verifyToken(AuthDto.TokenVerifyRequest request) {
     return webClient.post()
             .uri("/internal/v1/auth/verify")
             .bodyValue(request)
             .retrieve()
             .bodyToMono(new ParameterizedTypeReference<CommonResponse<AuthDto.TokenVerifyData>>() {})
+            .timeout(java.time.Duration.ofSeconds(3))
             .onErrorReturn(new CommonResponse<>(false, "인증 서비스 장애", new AuthDto.TokenVerifyData(false), null));
 }

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@src/main/java/org/pgsg/gateway/client/AuthClient.java` around lines 20 - 27,
The verifyToken method currently makes an external call via webClient.post()
without a timeout; add a reactor timeout to the returned Mono (e.g., append
.timeout(Duration.ofSeconds(<n>)) after bodyToMono(...)) and handle timeout
errors explicitly (use onErrorResume to convert TimeoutException or other errors
into the same CommonResponse fallback used now) so verifyToken returns a timely
failure instead of hanging; update references in this method (verifyToken,
webClient.post(), bodyToMono(...), onErrorReturn) accordingly.

src/main/java/org/pgsg/gateway/filter/JwtGatewayFilter.java (1)

182-187: ⚡ Quick win

tracer.currentSpan() 중복 호출 개선.

tracer.currentSpan()을 두 번 호출하고 있습니다. null 체크 이후 다시 호출하면 이론적으로 race condition이 발생할 수 있고, 불필요한 중복 호출입니다.

♻️ 로컬 변수로 개선

 private String resolveTraceId() {
-    if (tracer.currentSpan() != null) {
-        return Objects.requireNonNull(tracer.currentSpan()).context().traceId();
-    }
-    return UUID.randomUUID().toString().substring(0, 8);
+    var span = tracer.currentSpan();
+    if (span != null) {
+        return span.context().traceId();
+    }
+    return UUID.randomUUID().toString().substring(0, 8);
 }

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@src/main/java/org/pgsg/gateway/filter/JwtGatewayFilter.java` around lines 182
- 187, In resolveTraceId(), avoid calling tracer.currentSpan() twice by
assigning it to a local variable (e.g., Span span = tracer.currentSpan()), check
that variable for null, and if non-null return span.context().traceId();
otherwise return UUID.randomUUID().toString().substring(0,8); this removes the
duplicate call and eliminates the potential race condition while preserving
behavior.

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@build.gradle`:
- Around line 36-38: Fix the typos in the three Gradle exclude lines so
exclusions actually apply: remove the trailing colon from the group and strip
the version from the module in the org.springdoc exclude (use group
"org.springdoc" and module "springdoc-openapi-starter-webmvc-ui"), correct the
misspelled group "org.org.springframework.cloud" to "org.springframework.cloud"
for the spring-cloud exclude, and leave the spring-boot exclude group/module as
"org.springframework.boot" and "spring-boot-starter-web"; update the three
exclude(...) calls accordingly.

---

Nitpick comments:
In `@src/main/java/org/pgsg/gateway/client/AuthClient.java`:
- Line 10: Remove the leftover commented FeignClient annotation line in
AuthClient.java (the line containing //@FeignClient(name = "user-service",
fallbackFactory = AuthClientFallbackFactory.class)) since the project has
switched to WebClient; simply delete that commented line so the codebase no
longer contains unused Feign-related comments.
- Around line 20-27: The verifyToken method currently makes an external call via
webClient.post() without a timeout; add a reactor timeout to the returned Mono
(e.g., append .timeout(Duration.ofSeconds(<n>)) after bodyToMono(...)) and
handle timeout errors explicitly (use onErrorResume to convert TimeoutException
or other errors into the same CommonResponse fallback used now) so verifyToken
returns a timely failure instead of hanging; update references in this method
(verifyToken, webClient.post(), bodyToMono(...), onErrorReturn) accordingly.

In `@src/main/java/org/pgsg/gateway/filter/JwtGatewayFilter.java`:
- Around line 182-187: In resolveTraceId(), avoid calling tracer.currentSpan()
twice by assigning it to a local variable (e.g., Span span =
tracer.currentSpan()), check that variable for null, and if non-null return
span.context().traceId(); otherwise return
UUID.randomUUID().toString().substring(0,8); this removes the duplicate call and
eliminates the potential race condition while preserving behavior.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 0dd4166f-f008-4ef0-8d02-ed3acd48fe46

📥 Commits

Reviewing files that changed from the base of the PR and between e696ff1 and 396611a.

📒 Files selected for processing (13)

• build.gradle
• docker-compose.yaml
• src/main/java/org/pgsg/gateway/GatewayApplication.java
• src/main/java/org/pgsg/gateway/auth/AuthProvider.java
• src/main/java/org/pgsg/gateway/auth/AuthProviderImpl.java
• src/main/java/org/pgsg/gateway/client/AuthClient.java
• src/main/java/org/pgsg/gateway/config/GatewayAppCtx.java
• src/main/java/org/pgsg/gateway/config/GatewaySecurityConfig.java
• src/main/java/org/pgsg/gateway/feign/AuthClient.java
• src/main/java/org/pgsg/gateway/feign/AuthClientFallbackFactory.java
• src/main/java/org/pgsg/gateway/filter/HttpRequestHeaderWrapper.java
• src/main/java/org/pgsg/gateway/filter/JwtGatewayFilter.java
• src/test/java/org/pgsg/gateway/JwtGatewayIntegrationTest.java

💤 Files with no reviewable changes (5)

• docker-compose.yaml
• src/main/java/org/pgsg/gateway/filter/HttpRequestHeaderWrapper.java
• src/main/java/org/pgsg/gateway/feign/AuthClientFallbackFactory.java
• src/main/java/org/pgsg/gateway/GatewayApplication.java
• src/main/java/org/pgsg/gateway/feign/AuthClient.java