[BE] RequestWrappingFilter를 개선한다.

[seokjin8678]

✨ 세부 내용

RequestWrappingFilter 클래스는 다음과 같이 OncePerRequestFilter를 구현하고 있는 필터입니다.

@Component
public class RequestWrappingFilter extends OncePerRequestFilter {

    @Override
    protected void doFilterInternal(HttpServletRequest request,
                                    HttpServletResponse response, FilterChain chain)
        throws ServletException, IOException {

        ContentCachingRequestWrapper wrappingRequest = new ContentCachingRequestWrapper(request);

        chain.doFilter(wrappingRequest, response);
    }
}

해당 필터는 요청의 payload를 로깅하기 위해 HttpServletRequest을 ContentCachingRequestWrapper로 감싸 input stream을 재사용 할 수 있도록 하는 역할입니다.

하지만 현재 코드에서는 요청의 payload를 로깅하는 코드가 존재하지 않습니다.

따라서 해당 필터를 유지하는 것은 불필요한 성능 낭비를 유발합니다.

해당 필터를 유지하려고 한다면 특정 경로(@LogRequestBody 어노테이션이 붙은 핸들러 메서드의 경로)에 들어올 때만 ContentCachingRequestWrapper로 HttpServletRequest를 감싸면 성능 낭비를 줄일 수 있을 것 같습니다.

@Override
protected void doFilterInternal(
    HttpServletRequest request,
    HttpServletResponse response,
    FilterChain chain
) throws ServletException, IOException {
    if (fooMatcher.match(request.getMethod(), request.getRequestURI())) {
        ContentCachingRequestWrapper wrappingRequest = new ContentCachingRequestWrapper(request);
        chain.doFilter(wrappingRequest, response);
    } else {
        chain.doFilter(request, response);
    }
}

만약 개선을 하더라도 사용이 적거나 애매하다면, 삭제를 하는 방향으로 가도 좋을 것 같네요.

⏰ 예상 소요 시간

3시간