Skip to content

[hotfix] filter error #300

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
PicturePark1101 wants to merge 9 commits into from
Closed

[hotfix] filter error #300

Changes from all commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
a771e80
[deploy] dev > prod
2hy2on Dec 13, 2024
db9cea0
[deploy] dev > prod
2hy2on Dec 16, 2024
399d8d2
Merge pull request #264 from Juinjang/dev
PicturePark1101 Dec 18, 2024
fac5d25
Merge pull request #266 from Juinjang/dev
PicturePark1101 Dec 30, 2024
d169bbf
Merge pull request #284 from Juinjang/dev
PicturePark1101 Jan 20, 2025
3d4c276
[deploy] prod 서버에 반영
2hy2on Jan 20, 2025
3472e86
Merge pull request #292 from Juinjang/dev
PicturePark1101 Jan 23, 2025
9e52e88
[deploy] dev > prod
2hy2on Feb 6, 2025
6fe1945
hotfix : 로그필터에서 요청 캐싱 주석처리
PicturePark1101 Feb 6, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
42 changes: 23 additions & 19 deletions src/main/java/umc/th/juinjang/controller/monitoring/APILoggerFilter.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,10 @@
import static umc.th.juinjang.utils.LoggerProvider.registerRequestId;

import jakarta.servlet.FilterChain;
import jakarta.servlet.ServletException;
import jakarta.servlet.http.HttpServletRequest;
import jakarta.servlet.http.HttpServletResponse;
import java.io.IOException;
import java.util.List;
import java.util.UUID;
import lombok.extern.slf4j.Slf4j;
Expand All @@ -28,25 +30,27 @@ public APILoggerFilter(List<String> EXCLUDED_URLS) {
}

@Override
protected void doFilterInternal(HttpServletRequest servletRequest, HttpServletResponse servletResponse, FilterChain chain ) {
CustomContentCachingHttpRequestWrapper requestWrapper = new CustomContentCachingHttpRequestWrapper(servletRequest);
ContentCachingResponseWrapper responseWrapper = new ContentCachingResponseWrapper(servletResponse);
registerRequestId(UUID.randomUUID().toString());

try {
if (shouldNotFilter(requestWrapper)) {
chain.doFilter(requestWrapper, responseWrapper);
return;
}
apiLoggerPrinter.print(new APIRequestLoggerGenerator(requestWrapper));
chain.doFilter(requestWrapper, responseWrapper);
apiLoggerPrinter.print(new APIResponseLoggerGenerator(responseWrapper));
responseWrapper.copyBodyToResponse();
} catch (Exception e) {
logger.error("APILogger 필터 오류");
} finally {
MDC.clear();
}
protected void doFilterInternal(HttpServletRequest servletRequest, HttpServletResponse servletResponse, FilterChain chain )
throws ServletException, IOException {
// CustomContentCachingHttpRequestWrapper requestWrapper = new CustomContentCachingHttpRequestWrapper(servletRequest);
// ContentCachingResponseWrapper responseWrapper = new ContentCachingResponseWrapper(servletResponse);
// registerRequestId(UUID.randomUUID().toString());

chain.doFilter(servletRequest, servletResponse);
// try {
// if (shouldNotFilter(requestWrapper)) {
// chain.doFilter(requestWrapper, responseWrapper);
// return;
// }
// apiLoggerPrinter.print(new APIRequestLoggerGenerator(requestWrapper));
// chain.doFilter(requestWrapper, responseWrapper);
// apiLoggerPrinter.print(new APIResponseLoggerGenerator(responseWrapper));
// responseWrapper.copyBodyToResponse();
// } catch (Exception e) {
// logger.error("APILogger 필터 오류");
// } finally {
// MDC.clear();
// }
}

@Override
Expand Down