Skip to content
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.

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

feat: API 요청 정보와 실행 시간에 대한 로그 추가 #279

Merged
merged 5 commits into from
Nov 6, 2023

Conversation

leeeeeyeon
Copy link
Contributor

🛰️ Issue Number

#182

🪐 작업 내용

  • AOP를 적용하여 API 요청 정보와 실행 시간에 대한 로그를 추가하였습니다. 자세한 내용은 우측 포스팅을 참고해주세요. → 관련 포스팅
  • p6spy를 운영 환경에서 사용하는 것은 자원 소모가 크다고 하여 운영 환경에서의 옵션을 껐습니다.
  • 쿼리가 길어질 경우 p6spy 로그의 가독성이 떨어져 포맷터를 구현하였습니다.

👋 리뷰어가 중점적으로 확인해야 할 것 (Optional)

API 요청 정보와 실행 시간에 대한 로그는 아래와 같이 출력됩니다.
image

p6spy 쿼리 로그는 아래와 같이 출력됩니다.
image

🔬 테스트 코드 결과

image

✅ Check List

  • 코드가 정상적으로 컴파일되나요?
  • 테스트 코드를 통과했나요?
  • merge할 브랜치의 위치를 확인했나요?
  • Reviewer를 지정했나요?
  • Assignee를 지정했나요?
  • Label을 지정했나요?

@leeeeeyeon leeeeeyeon added the ☁️ feature New feature or request label Nov 6, 2023
@leeeeeyeon leeeeeyeon requested a review from yth01 November 6, 2023 14:47
@leeeeeyeon leeeeeyeon self-assigned this Nov 6, 2023
Copy link

codecov bot commented Nov 6, 2023

Codecov Report

Merging #279 (af3b83a) into develop (9cf84e9) will decrease coverage by 0.27%.
Report is 5 commits behind head on develop.
The diff coverage is 37.25%.

Impacted file tree graph

@@              Coverage Diff              @@
##             develop     #279      +/-   ##
=============================================
- Coverage      47.62%   47.36%   -0.27%     
- Complexity       369      378       +9     
=============================================
  Files            100      102       +2     
  Lines           1976     2027      +51     
  Branches          54       56       +2     
=============================================
+ Hits             941      960      +19     
- Misses          1017     1047      +30     
- Partials          18       20       +2     
Files Coverage Δ
...a/com/gsm/blabla/global/config/P6SpyFormatter.java 83.33% <83.33%> (ø)
...ain/java/com/gsm/blabla/global/util/LogAspect.java 23.07% <23.07%> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ee5b7c9...af3b83a. Read the comment docs.

Copy link

sonarqubecloud bot commented Nov 6, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@leeeeeyeon leeeeeyeon merged commit 83ecef5 into develop Nov 6, 2023
3 of 4 checks passed
yth01 pushed a commit to yth01/blabla-server that referenced this pull request Jan 22, 2024
* chore: 운영 환경에서 p6spy 로그 출력 off 처리

* chore: .gitignore에 properties 확장자 파일 추가

* refactor: p6spy 쿼리 로그에 멀티 라인 적용

* feat: AOP를 이용한 API 요청 정보 및 실행 시간 측정 로그 추가

* refactor: 코드 스멜 제거
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
☁️ feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant