[Fix] FirebaseSubscribeService의 토큰 검증 로직 변경 #133
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…n 방식에서 메시지를 전달해보는 방식으로 변경 verifyIdToken의 경우 클라이언트 측에서 별도의 ID 토큰을 생성하여 전달해주어야 하기 때문에 일단 기존의 방식으로 변경한다
|
zbqmgldjfh
added a commit
that referenced
this pull request
Feb 18, 2024
* [�Feat] Hexagonal architecture #129 (#131) * setting(archUnit): 아키텍처 테스팅 의존성 추가 * test(HexagonalArchitecture): User의 HexagonalArchitecture 검증 * refactor: User 도메인 헥사고날 아키텍처로 전환 * feat: 헥사고날 애너테이션 적용 * refactor: API가 포트에 의존하도록 변경 * refactor: UseCase의 구현체들이 Port에 의존하도록 리팩토링 * test: Notice의 HexagonalArchitecture 검증 * refactor: Notice 도메인 헥사고날 아키텍처로 전환 * test: Admin의 HexagonalArchitecture 검증 * refactor(NoticeUpdateSupport): NoticeUpdateSupport 일부 로직 수정 * refactor: Admin 도메인 헥사고날 아키텍처로 전환 * refactor: 필요없는 설정파일 삭제 * test: Staff의 HexagonalArchitecture 검증 * refactor: Staff 도메인 헥사고날 아키텍처로 전환 * feat: User 도메인의 Firebase 로직을 Event로 처리하도록 구현 * feat: User-Token을 preHandler로 검증하도록 변경 * feat(MessageUserEventListener): Token Validation 리스너 제거 * feat(FirebaseService): 비즈니스 로직이 port에 의존하도록 리팩토링 * feat(FirebaseWithAdminUseCase): 어드민의 공지 전송 작업을 이벤트로 구현 * refactor: 기존의 단일 FirebaseService 를 구독과 공지 전송 2개의 서비스로 분리한다 * refactor(DependencyRuleTests): 기존 아키텍처 검증에서 event 페키지 추가 검증하도록 구현 * fix: QueryDsl이 사용하는 dto의 경로 재설정 * chore: 테스트 서식 지정자 제거와 주석 제거 * docs(README): 문서 업데이트 * refactor(ServerProperties): 공통으로 사용중인 서버의 환경변수를 Common 패키지 하부로 이동 * refactor(FirebaseExceptionHandler): exception handler의 페키지를 adapter.out으로 이동 * [Fix] FirebaseSubscribeService의 토큰 검증 로직 변경 (#133) verifyIdToken의 경우 클라이언트 측에서 별도의 ID 토큰을 생성하여 전달해주어야 하기 때문에 일단 기존의 방식으로 변경한다 * fix(NoticeQueryApiV2#getNotices): 학과공지 조회에서 important필드 결여로 인한 문제 해결 완료 (#135) * [Feat] Api docs v2 #125 (#136) * setting: Swagger 의존성 추가 * feat: API 문서화 완료 * setting: Swagger, TestContainer 버전 변경 * setting: SpringBoot version migration from 2.5.5 to 2.7.18 * feat: 문서화에서 제외할 Controller들에게 Hidden 추가 * setting: 사용하지 않는 gradle의 과정 제거
zbqmgldjfh
added a commit
that referenced
this pull request
Feb 18, 2024
verifyIdToken의 경우 클라이언트 측에서 별도의 ID 토큰을 생성하여 전달해주어야 하기 때문에 일단 기존의 방식으로 변경한다
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
verifyIdToken의 경우 클라이언트 측에서 별도의 ID 토큰을 생성하여 전달해주어야 하기 때문에 일단 기존의 방식으로 변경한다