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

fix: 소셜 로그인 관련 긴급 에러 수정 #214

Closed
1 task done
choidongkuen opened this issue Feb 12, 2024 · 0 comments · Fixed by #215
Closed
1 task done

fix: 소셜 로그인 관련 긴급 에러 수정 #214

choidongkuen opened this issue Feb 12, 2024 · 0 comments · Fixed by #215
Assignees

Comments

@choidongkuen
Copy link
Member

choidongkuen commented Feb 12, 2024

기능

  • 릴리즈 된 로직에서 소셜 로그인 에러 해결

Todo

  • OAuth 소셜 로그인 관련 긴급 에러 수정
@choidongkuen choidongkuen self-assigned this Feb 12, 2024
choidongkuen added a commit that referenced this issue Feb 12, 2024
* fix: 충돌 해결 (#210)

* fix: 소셜 로그인 에러 긴급 수정 (#214) (#215)

---------

Co-authored-by: ddingmin <ddingminyt@gmail.com>
choidongkuen added a commit that referenced this issue Feb 13, 2024
* fix: 충돌 해결 (#210)

* fix: 소셜 로그인 에러 긴급 수정 (#214) (#215)

* feat: 기존 회원 리뷰 조회 API 로직 변경 (#218)

* feat: userId 에 해당하는 유저 리뷰 조회 API 로직 변경 (#217)

* test: 통합테스트 수정 (#217)

* test: 단위테스트 수정 (#217)

---------

Co-authored-by: ddingmin <ddingminyt@gmail.com>
devxb added a commit that referenced this issue Feb 15, 2024
* fix: 충돌 해결 (#210)

* fix: 소셜 로그인 에러 긴급 수정 (#214) (#215)

* feat: 기존 회원 리뷰 조회 API 로직 변경 (#218)

* feat: userId 에 해당하는 유저 리뷰 조회 API 로직 변경 (#217)

* test: 통합테스트 수정 (#217)

* test: 단위테스트 수정 (#217)

* docs: readme 작성 (#222)

* docs: readme 작성

* docs: readme 수정

* test: QA용 EndMeetingAlerted를 발행한다

---------

Co-authored-by: ddingmin <ddingminyt@gmail.com>
Co-authored-by: ChoiDongKuen <danaver12@daum.net>
devxb added a commit that referenced this issue Feb 15, 2024
* fix: 충돌 해결 (#210)

* fix: 소셜 로그인 에러 긴급 수정 (#214) (#215)

* feat: 기존 회원 리뷰 조회 API 로직 변경 (#218)

* feat: userId 에 해당하는 유저 리뷰 조회 API 로직 변경 (#217)

* test: 통합테스트 수정 (#217)

* test: 단위테스트 수정 (#217)

* docs: readme 작성 (#222)

* docs: readme 작성

* docs: readme 수정

* test: QA용 EndMeetingAlerted를 발행한다

* feat: 유저가 삭제되었을때, 알람토큰도 삭제하도록 수정 (#226)

* feat: 유저가 삭제되었을때, 알람토큰도 삭제하도록 수정한다

* test: ApplicationEventPublisher 바인딩 안되는 테스트 삭제

* refactor: UserDeletedEvent 핸들러 이름 변경

* test: 친구조회 테스트 수정

* refactor: code smell을 제거한다

---------

Co-authored-by: ddingmin <ddingminyt@gmail.com>
Co-authored-by: ChoiDongKuen <danaver12@daum.net>
devxb added a commit that referenced this issue Feb 16, 2024
* fix: 충돌 해결 (#210)

* fix: 소셜 로그인 에러 긴급 수정 (#214) (#215)

* feat: 기존 회원 리뷰 조회 API 로직 변경 (#218)

* feat: userId 에 해당하는 유저 리뷰 조회 API 로직 변경 (#217)

* test: 통합테스트 수정 (#217)

* test: 단위테스트 수정 (#217)

* docs: readme 작성 (#222)

* docs: readme 작성

* docs: readme 수정

* test: QA용 EndMeetingAlerted를 발행한다

* feat: 유저가 삭제되었을때, 알람토큰도 삭제하도록 수정 (#226)

* feat: 유저가 삭제되었을때, 알람토큰도 삭제하도록 수정한다

* test: ApplicationEventPublisher 바인딩 안되는 테스트 삭제

* refactor: UserDeletedEvent 핸들러 이름 변경

* test: 친구조회 테스트 수정

* refactor: code smell을 제거한다

* fix: 모임 시작, 종료 알림이 모임 참여자가 3명 이상이 아니라면, 발행되지않도록 수정 (#231)

* feat: 리뷰 등록시 모임 참여 유저 조회 API 수정 (#229)

* refactor: MeetingController getCurrentUserId() 메소드로 통일 (#228)

* feat: 리뷰에 자신 포함 확인 로직 추가 (#195)

* feat: 모임 참여자 자신 포함 여부 확인 로직 추가(#228)

* test: 단위 테스트 추가 (#228)

* test: 통합 테스트 수정 및 실패 케이스 추가 (#228)

* refactor: title, body alert data에 추가

---------

Co-authored-by: ddingmin <ddingminyt@gmail.com>
Co-authored-by: ChoiDongKuen <danaver12@daum.net>
choidongkuen added a commit that referenced this issue Feb 16, 2024
* fix: 충돌 해결 (#210)

* fix: 소셜 로그인 에러 긴급 수정 (#214) (#215)

* feat: 기존 회원 리뷰 조회 API 로직 변경 (#218)

* feat: userId 에 해당하는 유저 리뷰 조회 API 로직 변경 (#217)

* test: 통합테스트 수정 (#217)

* test: 단위테스트 수정 (#217)

* docs: readme 작성 (#222)

* docs: readme 작성

* docs: readme 수정

* test: QA용 EndMeetingAlerted를 발행한다

* feat: 유저가 삭제되었을때, 알람토큰도 삭제하도록 수정 (#226)

* feat: 유저가 삭제되었을때, 알람토큰도 삭제하도록 수정한다

* test: ApplicationEventPublisher 바인딩 안되는 테스트 삭제

* refactor: UserDeletedEvent 핸들러 이름 변경

* test: 친구조회 테스트 수정

* refactor: code smell을 제거한다

* fix: 모임 시작, 종료 알림이 모임 참여자가 3명 이상이 아니라면, 발행되지않도록 수정 (#231)

* feat: 리뷰 등록시 모임 참여 유저 조회 API 수정 (#229)

* refactor: MeetingController getCurrentUserId() 메소드로 통일 (#228)

* feat: 리뷰에 자신 포함 확인 로직 추가 (#195)

* feat: 모임 참여자 자신 포함 여부 확인 로직 추가(#228)

* test: 단위 테스트 추가 (#228)

* test: 통합 테스트 수정 및 실패 케이스 추가 (#228)

* refactor: title, body alert data에 추가

* feat: 리뷰 등록시 요청 DTO Validation 변경 및 로직 추가 (#233)

* feat: 요청 validation 변경 (#228)

* test: UserControllerTest 리팩토링 (#224)

* feat: 리뷰 등록시 userService 로직 추가 (#224)

* test: 테스트 데이터 관련 메소드 추가 (#224)

* test: 리뷰 등록시 로직 추가에 따른 통합 테스트 추가 (#224)

* test: 리뷰 등록시 로직 추가에 따른 Service 단위 테스트 추가 (#224)

* feat: Cors 허용 메소드 추가 (#224)

---------

Co-authored-by: ddingmin <ddingminyt@gmail.com>
Co-authored-by: devxb <dlwnsdud205@naver.com>
Co-authored-by: xb205 <62425964+devxb@users.noreply.github.com>
ddingmin added a commit that referenced this issue Feb 16, 2024
* fix: 충돌 해결 (#210)

* fix: 소셜 로그인 에러 긴급 수정 (#214) (#215)

* feat: 기존 회원 리뷰 조회 API 로직 변경 (#218)

* feat: userId 에 해당하는 유저 리뷰 조회 API 로직 변경 (#217)

* test: 통합테스트 수정 (#217)

* test: 단위테스트 수정 (#217)

* docs: readme 작성 (#222)

* docs: readme 작성

* docs: readme 수정

* test: QA용 EndMeetingAlerted를 발행한다

* feat: 유저가 삭제되었을때, 알람토큰도 삭제하도록 수정 (#226)

* feat: 유저가 삭제되었을때, 알람토큰도 삭제하도록 수정한다

* test: ApplicationEventPublisher 바인딩 안되는 테스트 삭제

* refactor: UserDeletedEvent 핸들러 이름 변경

* test: 친구조회 테스트 수정

* refactor: code smell을 제거한다

* fix: 모임 시작, 종료 알림이 모임 참여자가 3명 이상이 아니라면, 발행되지않도록 수정 (#231)

* feat: 리뷰 등록시 모임 참여 유저 조회 API 수정 (#229)

* refactor: MeetingController getCurrentUserId() 메소드로 통일 (#228)

* feat: 리뷰에 자신 포함 확인 로직 추가 (#195)

* feat: 모임 참여자 자신 포함 여부 확인 로직 추가(#228)

* test: 단위 테스트 추가 (#228)

* test: 통합 테스트 수정 및 실패 케이스 추가 (#228)

* refactor: title, body alert data에 추가

* feat: 리뷰 등록시 요청 DTO Validation 변경 및 로직 추가 (#233)

* feat: 요청 validation 변경 (#228)

* test: UserControllerTest 리팩토링 (#224)

* feat: 리뷰 등록시 userService 로직 추가 (#224)

* test: 테스트 데이터 관련 메소드 추가 (#224)

* test: 리뷰 등록시 로직 추가에 따른 통합 테스트 추가 (#224)

* test: 리뷰 등록시 로직 추가에 따른 Service 단위 테스트 추가 (#224)

* feat: Cors 허용 메소드 추가 (#224)

* feat: 위치 기반 API Gatling 부하 테스트 (#220)

* chore: docker-compose.yml 작성 (#195)

* test: Gatling 부하 테스트 구현 (#195)

* Update .env

* Delete .env

---------

Co-authored-by: ChoiDongKuen <danaver12@daum.net>
Co-authored-by: devxb <dlwnsdud205@naver.com>
Co-authored-by: xb205 <62425964+devxb@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant