[fix] 권한 제거 #407
Merged
[fix] 권한 제거 #407
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
[fix] price null
[refactor] 인증코드 확인 api 분리
* #123 feat: sse 전송 문제 해결 - SseEmitterKey 객체 추가 - SseEmitterManager 객체의 해시맵의 키 타입을 SseEmitterKey로 변경 * #123 fix: catch문 오타 수정 * #123 fix: sseEmitter가 리스너에서 예외 발생시 sseEmitterManager에서 emitter가 줄어들지 않는 문제 해결 - PortfolioEventListener에서 예외 발생시 해당 SseEmitter는 해시맵에서 제거되도록 변경하여 sseEmitter가 줄어들지 않는 문제를 해겷하였습니다. * #123 test: 오타 수정 * #123 fix: 사용하지 않는 createRedisKey 메소드 제거 * #122 feat: 에상 배당금 메소드 구현 - 현재 년도의 배당금이 없는경우 작년도의 배당금 정보를 이용하여 예상 배당금을 계사하는 메소드를 구현하였습니다 * #122 feat: 예상 배당금 계산 메소드에 대한 테스트 코드 구현 * #122 feat: 포트폴리오 종목에 대한 예상 연간배당금 및 예상 연간배당금 비율 계산 메소드 구현 * #122 fix: 현재년도를 가지고 있는 LocalDate 데이터를 매개변수화 * #122 fix: 테스트 코드 오류 수정 * #122 feat: 포트폴리오의 연간배당금 계산 메소드 변경 * #122 test: 테스트 코드 단언문 변경 - 좀더 상세한 결과를 검증하도록 변경 * #122 docs: 포트폴리오 샘플 데이터에 create_at 컬럼 데이터 추가 * [feat] 포트폴리오 추가 및 수정시 입력정보 변경 (#152) * #144 fix: 데이터 유효성 범위 변경 - 예산, 수익금액, 최대손실 금액 정보에 대하여 0또한 포함이 가능하도록 변경 * #144 test: 테스트 코드 수정 * #144 fix: 검증 조건 변경 * #144 test: 테스트 코드 직렬화/역직렬화 코드 수정 * #144 fix: 포트폴리오 수정 요청 리퀘스트의 데이터 유효성 변경 * #144 test: 테스트 코드 수정 - 입력값 파라미터화 * #144 feat: 메일전송여부를 관리하는 매니저 객체 생성 * #144 feat: 예산이 0원인지 검사하는 메소드 추가 * #144 test: 목표 수익 알림 메소드에 대한 예외 케이스 추가 * #144 test: 최대손실알림 메일 스케줄링 메소드 테스트 코드 추가
- 데이터베이스에 반영하도록 변경
* #164 test: 프로필 변경 테스트 코드 작성 * #162 fix: 메소드명 변경 - 실제 이미지 파일이 아닌 url값을 변경하는 것이기 때문에 updateProfileUrl로 변경함 * #164 fix: 불필요한 update jpql 제거 - Member 더티 체킹을 통해서 작동하기 때문에 제거함 * #164 feat: 프로필 변경 관련 에러 코드 추가 - 클라이언트로부터 변경할 정보를 받지 못한 경우 작동될 수 있음 * #164 refactor: 프로필 변경 서비스 리팩토링 - Optional을 통한 Null처리 수행함 - 닉네임 유효성 검사를 ProfileChangeRequest로 이전 - Optional Method Chain을 이용하여 최대한 가독성있게 변경 - extract 메소드 실행 과정중에 검증 수행 - application.yml 파일에 multipart file size를 명시하여 이미지가 너무 큰 경우 받을 수 없도록 명세함 * #164 fix: 공백 생성자 및 getter 추가 * #164 test: 프로필 변경 컨트롤러 테스트 코드 추가 * #164 fix: @Valid 애노테이션 추가 * #164 test: 프로필 변경 서비스 에외 테스트 추가 - 닉네임이 중복되는 경우 테스트 추가
* #167 test: 일반 회원가입 컨트롤러 테스트 코드 구현 * #167 fix: 회원 에러 코드 메시지 오타 수정 * #167 fix: 회원 성공 코드 메시지 오타 수정 * #167 refactor: 일반 회원가입 컨트롤러 및 서비스 코드 리팩토링 - SignUpServiceRequest 추가 - 일반 회원가입 서비스에서 검증 코드 정리 * #167 feat: MissingServletRequestPartException 글로벌 핸들러 추가 * #167 test: 일반 회원가입 컨트롤러 테스트 코드 구현 - 프로필 없이 일반 회원가입 성공 - signupData 필드없이 회원가입 실패 * #167 test: 일반 회원가입 서비스 테스트 코드 구현 - 일반 회원가입 서비스 성공 - 닉네임, 이메일 중복되는 경우 예외 테스트 - 비밀번호 불일치 예외 테스트 * #167 test: 회원가입 서비스 예외 케이스 테스트 코드 추가 - 프로필 이미지 사이즈가 초과하는 경우 회원가입 불가능
* #167 test: 테스트 코드 실패 수정 - currentPriceManager 목객체를 추가하여 redis를 모킹 처리함 * #121 test: @beforeeach에 배당금 제거 추가
* #167 test: 일반 회원가입 컨트롤러 테스트 코드 구현 * #167 fix: 회원 에러 코드 메시지 오타 수정 * #167 fix: 회원 성공 코드 메시지 오타 수정 * #167 refactor: 일반 회원가입 컨트롤러 및 서비스 코드 리팩토링 - SignUpServiceRequest 추가 - 일반 회원가입 서비스에서 검증 코드 정리 * #167 feat: MissingServletRequestPartException 글로벌 핸들러 추가 * #167 test: 일반 회원가입 컨트롤러 테스트 코드 구현 - 프로필 없이 일반 회원가입 성공 - signupData 필드없이 회원가입 실패 * #167 test: 일반 회원가입 서비스 테스트 코드 구현 - 일반 회원가입 서비스 성공 - 닉네임, 이메일 중복되는 경우 예외 테스트 - 비밀번호 불일치 예외 테스트 * #167 test: 회원가입 서비스 예외 케이스 테스트 코드 추가 - 프로필 이미지 사이즈가 초과하는 경우 회원가입 불가능 * #167 test: 닉네임 중복 검사 서비스 테스트 코드 추가 * #167 test: 닉네임 중복 검사 컨트롤러 테스트 코드 추가 * #167 test: 이메일 중복 검사 테스트 코드 추가 * #167 test: 이메일 중복 검사 컨트롤러 테스트 코드 추가 * #167 refactor: 유효성 검사 추가 및 검증코드 생성기 추가 * #167 test: 검증 코드 컨트롤러 및 서비스 테스트 코드 추가 * #167 feat: 검증코드 요청에 대한 유효성 검사 추가 * #167 test: 검증 코드 검사에 대한 컨트롤러 및 서비스 테스트 코드 추가
[feat] change watchlist name
…trollerTest.java Co-authored-by: YongHwan Kim <dragonbead95@naver.com>
Co-authored-by: YongHwan Kim <dragonbead95@naver.com>
…SuccessCode.java Co-authored-by: YongHwan Kim <dragonbead95@naver.com>
…troller.java Co-authored-by: YongHwan Kim <dragonbead95@naver.com>
* feat: 액세스 토큰 갱신 롤백 * #371 fix: oauth 설정 오타 수정
* fix: method source 수정 * #371 fix: 토큰 갱신 문제 해결 - 필터를 대상으로 스프링 빈 등록으로 인하여 자동 등록됨
- PUT 메서드 방식 추가
* #350 refactor: List<NotificationCondition>을 제네릭 및 캡슐화 * #350 refactor: 목표 수익률에 대한 알림 전송 리팩토링 * #350 refactor: FirebaseNotificationProvider 추상화 - FirebasePortfolioNotificationProvider 구현 - FirebaseTargetPriceNotificationProvider 구현 * #350 test: 최대 손실율 달성 알림 테스트 추가 * #350 rename: 변수명 변경 * #350 refactor: notifyTargetGain 메서드 리팩토링 * #350 style: saveNotification 메서드 코드 정리 * #350 refactor: NotificationService의 알림 전송 기능 Notifiable 타입으로 추상화 - Notifiable 인터페이스 정의 및 Portfolio 클래스가 구현 * #350 refactor: TargetPriceNotification 추상화 * #350 refactor: Service의 리턴타입을 추상화 * #350 test: 최대 손실율 알림 테스트 추가 * #350 refactor: NotificationDispatcher 추가 - 해당 객체는 여러 NotificationProvider를 실행하여 다양한 알림 매체에게 알림 전송을 요청합니다.(ex, fcm, kakao, mail...) * #350 style: code 정리 * #350 style: 코드 정리
* #390 feat: 포트폴리오 종목 리소스 owner 검증문 구현 * test: 테스트 코드 수정 * fix: 매입 이력 생성 리스폰스 수정 * feat: 관심종목 단일 삭제 API 추가 * feat: 종목 지정가 단일 제거 API 추가 * fix: stock, sector exception case 추가 * fix: 권한 제거
* #390 feat: 포트폴리오 종목 리소스 owner 검증문 구현 * test: 테스트 코드 수정 * fix: 매입 이력 생성 리스폰스 수정 * feat: 관심종목 단일 삭제 API 추가 * feat: 종목 지정가 단일 제거 API 추가 * fix: stock, sector exception case 추가 * fix: 권한 제거 * docs: Rest Docs 오타 수정
* build: QueryDSL 의존성 추가 * feat: getSliceOfStock * test: 종목 검색 테스트 추가 * test: 종목 스크롤 검색 Rest Docs 추가 * docs: 종목 스크롤 검색 Rest Dcos 설명문 추가
* test: 포트폴리오 삭제 권한 테스트 추가 * test: 포트폴리오 다수 제거 서비스의 권한 예외 테스트 추가 * test: 포트폴리오 종목 조회시 권한 예외 케이스 추가 * feat: 매입 이력 리소스 소유 권한 검증문 추가 * test: 매입 이력 추가시 권한 검증문 추가 * test: 매입이력 수정 및 삭제 서비스의 권한 검증문 추가 * test: 포트폴리오 차트 조회에 리소스 소유 검증문 추가 * feat: 권한 검증문 추가 * test: 포트폴리오 알림 상태 변경의 권한 검증 테스트 추가 * feat: 종목 지정가 단일 제거 서비스엣 권한 검증문 추가 * test: 종목 지정가 단일 제거 권한 테스트 추가 * feat: 회원 알림 서비스에 권한 검증문 추가 * test: 회원 알림의 권한 관련 테스트 추가 * feat: 관심종목 서비스에 권한 검증문 추가 * test: 관심종목 권한 테스트 추가
* test: 포트폴리오 삭제 권한 테스트 추가 * test: 포트폴리오 다수 제거 서비스의 권한 예외 테스트 추가 * test: 포트폴리오 종목 조회시 권한 예외 케이스 추가 * feat: 매입 이력 리소스 소유 권한 검증문 추가 * test: 매입 이력 추가시 권한 검증문 추가 * test: 매입이력 수정 및 삭제 서비스의 권한 검증문 추가 * test: 포트폴리오 차트 조회에 리소스 소유 검증문 추가 * feat: 권한 검증문 추가 * test: 포트폴리오 알림 상태 변경의 권한 검증 테스트 추가 * feat: 종목 지정가 단일 제거 서비스엣 권한 검증문 추가 * test: 종목 지정가 단일 제거 권한 테스트 추가 * feat: 회원 알림 서비스에 권한 검증문 추가 * test: 회원 알림의 권한 관련 테스트 추가 * feat: 관심종목 서비스에 권한 검증문 추가 * test: 관심종목 권한 테스트 추가 * fix: 테스트 실패 해결
* fix: 종목 지정가 알림 문제 해결
- 종목 지정가 알림 발송시 referenceId를 저장하는데 종목 지정가의 referenceId는
tickerSymbol로 저장되어서 policy 검사에서 계속 통과됨
* test: 종목 지정가 시나리오 추가
* fix: messageIdMap 생성 방식 변경
- key값으로 기존 referenceId에서 idToSentHistory 값으로 변경
- idToSentHistory 형식
- 종목 지정가 : targetPriceNotification:{targetPriceNotificationId}
- 포트폴리오 : portfolioNotification:{referenceId}
* refactor: refactor resource authorized * refactor: refactoring AuthorizationAspect \n - ResourceId annotation 추가 \n - 테스트 코드 실패 해결 \n - ResourceIdAspect 추가 * refactor: 리소스 id 컬렉션에 대한 검증문 리팩토링 - 서비스 매개변수로 리소스 등록번호 컬렉션 전달시 각각의 리소스에 대하여 소유 권한 여부를 검증하도록 리팩토링 * feat: findPortfolio 메서드에 리소스 소유 권한 검증 애노테이션 추가 * feat: PortfolioHoldingService에 리소스 소유 권한 검증 애노테이션 추가 * feat: 리소스 소유 권한 검증 애노테이션 추가 * refactor: annotation에 AuthorizeService 구현체 이름을 전달하는 방식으로 변경 * test: 테스트 실패 해결 - aop 적용에 따른 예외 케이스의 메시지 변경 * test: 테스트 실패 해결 - serviceClass 타입 오타 * feat: PurchaseHistoryService에 @authorize 애노테이션 적용 * test: 매입이력 서비스 테스트 실패 해결 * rename: AuthorizeService 이름 변경 * move: AuthorizedService 구현체 이동 * feat: fcm, portfolio 도메인에 @Authorized 애노테이션 추가 * test: 종목 지정가 테스트 실패 해결 * feat: 회원 알림 서비스에 리소스 소유 검증 애노테이션 추가 * feat: 관심 종목 리소스 소유 검증 애노테이션 추가
* refactor: refactor resource authorized * refactor: refactoring AuthorizationAspect \n - ResourceId annotation 추가 \n - 테스트 코드 실패 해결 \n - ResourceIdAspect 추가 * refactor: 리소스 id 컬렉션에 대한 검증문 리팩토링 - 서비스 매개변수로 리소스 등록번호 컬렉션 전달시 각각의 리소스에 대하여 소유 권한 여부를 검증하도록 리팩토링 * feat: findPortfolio 메서드에 리소스 소유 권한 검증 애노테이션 추가 * feat: PortfolioHoldingService에 리소스 소유 권한 검증 애노테이션 추가 * feat: 리소스 소유 권한 검증 애노테이션 추가 * refactor: annotation에 AuthorizeService 구현체 이름을 전달하는 방식으로 변경 * test: 테스트 실패 해결 - aop 적용에 따른 예외 케이스의 메시지 변경 * test: 테스트 실패 해결 - serviceClass 타입 오타 * feat: PurchaseHistoryService에 @authorize 애노테이션 적용 * test: 매입이력 서비스 테스트 실패 해결 * rename: AuthorizeService 이름 변경 * move: AuthorizedService 구현체 이동 * feat: fcm, portfolio 도메인에 @Authorized 애노테이션 추가 * test: 종목 지정가 테스트 실패 해결 * feat: 회원 알림 서비스에 리소스 소유 검증 애노테이션 추가 * feat: 관심 종목 리소스 소유 검증 애노테이션 추가 * move: 종목 지정가와 지정가 알림 분리 * docs: rest docs 목차 정리 * docs: rest docs 목차 정리
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.
변경점