Skip to content

hw1 #5

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

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

hw1 #5

wants to merge 10 commits into from

Conversation

qqwwee771441
Copy link

@qqwwee771441 qqwwee771441 commented Apr 8, 2025

변경점 👍

새로 구현한 기능 및 주요 변경점

  1. common
  • 베이스 엔티티, ULID
  • 익셉션, 익셉션 핸들러, 에러코드, 에러 리스폰스
  • 커먼 리스폰스
  1. documentation
  • GlobalApiResponse 명세서
  • 스와거 컨피그, 스와거 그룹 컨피그
  • auth, comment, post, user domain들에 대해서 명세서
  • 컨트롤러 태그들
  1. domain
  • auth (자체로그인)
  • oauth2 (소셜로그인)(OAuth2PkceController와 OAuth2PkceService는 사용안함)
  • user 유저
  • post 게시글
  • comment 댓글
  1. global
  • annotation 컨트롤러 매개변수에 특정 값 주입하는 어노테이션
  • jpa config, web config
  • resolver 어노테이션 동작 구현한 리졸버
  • security 설정, 필터, 인터셉터, jwt, attributes, userdetailsSerivce 등
  1. infrastructure
  • config 레디스에 대한 설정
  • repository 레디스 사용 인터페이스
  1. support
  • 실행마다 레디스 초기화 로직, 로컬환경일때만 실행됨.

버그 해결 💊

해결한 버그

  1. 테스트 코드 계속 오류 걸려서 java파일만 생성하고 내용은 다 지웠음.
  • 통합 테스트로 하니 잘됨
  • 슬라이스 테스트에서 Signup 테스트와 Login 테스트는 성공햇음.
  • Refresh 테스트와 Logout 테스트는 @InjectToken 어노테이션 때문에 오류 걸리는 것으로 보임.
  1. 스웨거 접속 오류는 익셉션 핸들러의 @RestControllerAdvice 를 주석 처리하니 잘 접속됨.

리팩토링 🛠

리팩토링 내역

  1. 기존에 컨트롤러에 작성했던 스웨거 관련 어노테이션들을 documentation.domain으로 옮겼고, 기존의 컨트롤러에는 @~~~Doc 이라는 애너테이션 한번만 추가해서 깔끔하게 코드를 수정함. -> 스웨거 코드와 로직 코드와 분리

  2. application.properties와 application-local.properties와 application-secret.properties를 분리하였음. application-secret.properties는 .gitignore에 추가하였음.

스크린샷 🖼

변경된 부분에 대한 스크린샷

auth
pkce
user
post
comment

비고 ✏

리뷰어에게 전하는 말 등

감사합니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant