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.

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

Jump to bottom

[fix #191] 일부 회원 인증/인가 실패 #195

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

[fix #191] 일부 회원 인증/인가 실패 #195

wants to merge 10 commits into from

Conversation

dudxo
Copy link
Collaborator

@dudxo dudxo commented Jan 14, 2025

관련 이슈

📑 작업 상세 내용

  • 일부 인원 회원 인증/인가 실패
    • 예상 오류 지점
        1. 쿠키 생성
        1. 클라이언트 쿠키 저장
        1. 클라이언트 저장된 쿠키 해독 불가
    • 오류 지점 확인을 위한 임시 로그 추가

💫 작업 요약

🔍 중점적으로 리뷰 할 부분

@dudxo dudxo added the 🚨 fix 오류 수정 label Jan 14, 2025
@dudxo dudxo self-assigned this Jan 14, 2025
@dudxo dudxo linked an issue Jan 14, 2025 that may be closed by this pull request
🚨 인증/인가 실패 #191
Open
hyun2371 and others added 3 commits January 14, 2025 20:58
@hyun2371 @dudxo
[refactor #193] 채팅 요청 상태 업데이트 함수 수정 (#194)
335eab1 
* [rename] : DTO 이름 변경

* [refactor] : 만료된 채팅 요청 아이디를 비교해 상태 변경하도록 수정

* [test] : 아이디를 비교해 상태 변경하도록 수정 -> 테스트 반영

* [refactor] : 스케줄러 함수명 변경

* [feat] : 만료된 채팅 요청 아이디로 벌크 연산 호출

* [test] : 만료된 채팅 요청 아이디로 벌크 연산 -> 테스트 반영

* [refactor] : 로직 변경으로 불필요해진 repository 함수 제거
@hyun2371 @dudxo
[fix #190] 답변자가 스스로에게 채팅 요청 보낼 수 없도록 수정 (#192)
5902032 
* [feat] : 채팅 요청 에러 코드 추가

* [feat] : 채팅 요청 에러 검증 로직 추가

* [test] : 채팅 요청 에러 검증 로직 테스트

* [refactor] : 검증 로직 함수 추출
@dudxo
[chore #191] 인증/인가 임시 로그 추가를 위한 CD 스크립트 수정
d20cca4
@github-actions GitHub Actions
Copy link

github-actions bot commented Jan 14, 2025 •
edited
Loading

Code Coverage

Overall Project 84.5% -0.11% 🍏
Files changed 93.33% 🍏

File Coverage
ChatInquiryService.java 94.12% 🍏
ChatInquiryQueryRepositoryImpl.java 72.06% -1.47% ❌
ChatInquiryScheduler.java 0% -80% ❌

@github-actions GitHub Actions
Copy link

github-actions bot commented Jan 14, 2025 •
edited
Loading

Test Results

 28 files   28 suites   13s ⏱️
143 tests 141 ✅ 2 💤 0 ❌
144 runs  142 ✅ 2 💤 0 ❌

Results for commit fac440d.

♻️ This comment has been updated with latest results.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@dudxo dudxo

Labels
🚨 fix 오류 수정
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

🚨 인증/인가 실패
2 participants
@dudxo @hyun2371