Feature/89 스웨거 API 테스트를 위한 봉사자 / 기관 토큰 발급 #91
Merged
Conversation
This file contains hidden or 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
3 tasks
- Authorization 없다면 익명 유저 - Authorization 있다면 Bearer 인증 작업
leebs0521
force-pushed
the
feature/89-add-test-token-publish
branch
from
c1d2ca8 to
fbc9de7
Compare
|
m-a-king
reviewed
Nov 30, 2024
Comment on lines
+50
to
56
| if (!accessToken.startsWith("Bearer ")) { | ||
| throw new JwtException(JwtErrorType.MISSING_TOKEN); | ||
| } | ||
|
|
||
| accessToken = accessToken.substring(7); | ||
|
|
||
| return new EncodedToken(accessToken); |
Collaborator
There was a problem hiding this comment.
이 부분은 제가 더 직관적으로 수정해보겠습니다! 그대로 올려주셔도 무방합니다.
m-a-king
approved these changes
Nov 30, 2024
7zrv
approved these changes
Nov 30, 2024
ayoung-dev
added a commit
that referenced
this pull request
Nov 30, 2024
* fix(community): CommunityComment 엔티티 및 Dto에 communityBoardId 추가 * feat(community): CommunityBoardRepository에 existsById 추가 및 검증 테스트 추가 * feat(community): CommunityComment 생성 시 게시글 검증 로직 추가 * test(community): CommunityComment 생성 시 게시글 검증 테스트 추가 * test(community): CommunityCommentRepositoryTest 수정 * test(community): 불필요한 import 및 변수 삭제 * Feature/75 관심기관 등록, 취소 기능 구현 (#85) * feat: 관심기관 등록 기능 구현 - InterestCenter 엔티티에 누락된 기관, 봉사자 id 필드 추가 - 관심 기관 등록 유스케이스 추가및 서비스 레이어 구현 - 등록, 검증을 위한 영속성 레이어 구현 - 중복 예외 클래스 추가 - 예외 메세지 추가 - 테스트 코드 작성및 검증 완 * feat: 관심기관 취소 기능 구현 - 관심 기관 취소 유스케이스 추가및 서비스 레이어 구현 - 관심기관 취소, 검증을 위한 영속성 레이어 구현 - 예외 메세지 추가 - 테스트 코드 작성및 검증 완료 * feat: 관심기관 API 컨트롤러 구현 - 관심 기관 등록과 취소 엔드포인트 컨트롤러 구현 - ApiResponse 공통 응답 객체 메서드 추가(논의 예정) - 테스트 코드 작성및 검증완료 * feat: 요청 유효성 검증추가 - 관심기관 등록 요청 Dto에 유효성 검증 어노테이션 추가 - 활성화를 위해 컨트롤러에 @Valid 어노테이션 추가 * refactor: 공통 응답 객체 수정및 적용 - 기존 와일드카드를 사용하던 ApiResponse의 ok 메서드를 String 타입으로 리팩토링 - 변경에 따른 컨트롤러 return문 수정 * chore: 요청 Dto swagger scheme 수정 - 봉사자 -> 기관으로 수정 * chore: 컨트롤러 엔드 포인트 수정 - 기존 RequestMapping을 이용한 전역적인 엔드포인트 적용에서 엔드포인트마다 url을 붙여주는 방식으로 수정 * cicd: workflow 환경변수 추가 (#90) - 기본 이미지 주소를 저장한 환경변수 값추가 * cicd: workflow 환경변수 추가 (#92) - swagger에서의 테스트를 위한 환경변수 추가 * feat: 스웨거 API 테스트를 위한 봉사자 / 기관 토큰 발급 (#91) * feat(jwt): JwtAuthFilter 필터 로직 수정 - Authorization 없다면 익명 유저 - Authorization 있다면 Bearer 인증 작업 * feat(swagger): 토큰 인증 추가 * feat(center): 이름 기반 조회 기능 추가 * feat(security): 디버깅을 위한 기관 봉사 토큰 발급 기능 * chore: 디버깅 env 추가 * chore: 불필요한 import 제거 * hotfix: fix typo (#94) * fix(community): CommunityComment 엔티티 및 Dto에 communityBoardId 추가 * feat(community): CommunityBoardRepository에 existsById 추가 및 검증 테스트 추가 * feat(community): CommunityComment 생성 시 게시글 검증 로직 추가 * test(community): CommunityComment 생성 시 게시글 검증 테스트 추가 * test(community): CommunityCommentRepositoryTest 수정 * test(community): 불필요한 import 및 변수 삭제 * refactor(community): 코드 리뷰 사항 반영 - existsById 메서드 QeuryDSL -> JPA - repositoryTest 빌더 beforeEach로 추출 - serviceTest 예외 메세지 static import - serviceTest usecase 주입 > repository로 주입 * test(community): 변수명 변경 --------- Co-authored-by: seojin Yoon <[email protected]> Co-authored-by: Beom-Su Lee <[email protected]>
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.
resolved :
📌 과제 설명
👩💻 요구 사항과 구현 내용
✅ PR 포인트 & 궁금한 점