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

refactor: 피드백 검증이 이뤄지지 않는 문제를 해결한다. #446

Merged
merged 18 commits into from
Jan 24, 2024

Conversation

kong-hana01
Copy link
Collaborator

@kong-hana01 kong-hana01 commented Dec 10, 2023

관련 이슈번호

작업 사항

  • 코드 한번 쭉 둘러보다가 피드백 검증이 제대로 이뤄지지 않는다는 부분을 확인해 이를 수정했습니다.
    • 검증이 되도록 수정
    • 테스트 코드 추가
  • 컨벤션 유지
    • final 키워드 추가
    • 코드 포맷팅

thdwoqor and others added 13 commits September 21, 2023 15:02
* feat: id를 사용하는 entity는 BaseEntity를 상속하도록 수정

* refactor: flyway 사용 규칙에 따라 스크립트 수정
* refactor: 패키지 및 클래스 명 수정

* feat: 카카오 소셜로그인 idToken 검증 및 payload 추출 기능 구현

* feat: 카카오 소셜로그인 기능 구현

* refactor: jwkProvider를 static 변수로 가지도록 수정

* refactor: 구글 소셜로그인 검증 방식 통일

* refactor: 구글 소셜로그인 검증 방식 추상화

* feat: 멤버 객체 내 nickname 필드 추가

* feat: 닉네임 객체 구현

* refactor: 클래스 네이밍 변경

* feat: JwkProvider를 찾지 못할 때 예외 구현 및 record -> class로 변경

* refactor: final 키워드 추가

* chore: 닉네임 추가 sql 버전 변경 5.3 -> 5.4

---------

Co-authored-by: thdwoqor <celebarte01@gmail.com>
* feat: IdTokenResolver가 Provider에 맞게 동작한다

* test: 지원하지 않는 Issuer 예외 테스트 추가

* refactor: 유지보수가 용이하도록 구조 개선

* refactor: 누락된 final 추가
* feat: 게스트로 로그인한다

* feat: 빈 패스워드는 어떤 문자열과도 일치하지 않는다

* refactor: Member 클래스 내 생성자를 정적 팩토리 메소드로 변경

* refactor: final 키워드 추가

* fix: 머지 중 누락

---------

Co-authored-by: Kong-hana01 <gksqlsl11@khu.ac.kr>
@kong-hana01 kong-hana01 added backend 백엔드 분야 fix 버그 수정 labels Dec 10, 2023
@kong-hana01 kong-hana01 self-assigned this Dec 10, 2023
@kong-hana01
Copy link
Collaborator Author

kong-hana01 commented Dec 14, 2023

CI 파일이 터지네요 ㅠㅠ

이거 현재 운영 중인 서버들로 바꿔볼게용

@kong-hana01
Copy link
Collaborator Author

@thdwoqor 파워 혹시 운영 서버에 깃허브 액션 러너 설치되어 있나용?

@kong-hana01 kong-hana01 merged commit cf8a1a1 into backend Jan 24, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend 백엔드 분야 fix 버그 수정
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants