[fix] property 오타 수정 #406

yonghwankim-dev · 2024-07-23T05:51:27Z

구현한 것

db connection property 오타 수정

[fix] price null

[refactor] 인증코드 확인 api 분리

* #149 feat: 비밀번호 변경 api 구현 * #149 feat: 로그아웃 후 계정 삭제 * #149 feat: 프로필 정보 변경 기능 구현 * #149 fix: controller 메소드 명 변경

* #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: 최대손실알림 메일 스케줄링 메소드 테스트 코드 추가

- 데이터베이스에 반영하도록 변경

@Valid

* #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>

- 액세스 및 리프레시 토큰 갱신하는 경우에만 리스폰스에 쿠키 설정하도록 변경

* #371 fix: 액세스 토큰 갱신 수정 - 액세스 및 리프레시 토큰 갱신하는 경우에만 리스폰스에 쿠키 설정하도록 변경 * #371 fix: 액세스 토큰 갱신 수정 - addHeader -> setHeader로 수정 - 하나의 Set-Cookie 헤더에 accessToken 및 refreshToken를 세미콜론으로 구분하여 저장 * #371 test: 테스트 코드 해결

* feat: 액세스 토큰 갱신 롤백 * #371 fix: oauth 설정 오타 수정

* feat: 액세스 토큰 갱신 롤백 * #371 fix: oauth 설정 오타 수정 * #371 fix: oauth 설정 오타 수정 * #371 fix: oauth 설정 오타 수정

* fix: method source 수정 * #371 fix: 토큰 갱신 문제 해결 - 필터를 대상으로 스프링 빈 등록으로 인하여 자동 등록됨

* #384 fix: 로그인 실패시 400 응답되도록 수정 * #384 test: 로그인 예외 케이스 실패 해결

- PUT 메서드 방식 추가

* #386 fix: cors 설정에서 오타 수정 - PUT 메서드 방식 추가 * #386 test: 엑세스 토큰 갱신 테스트 2개 임시 비활성화

* #386 test: 액세스 토큰 갱신 테스트 코드 문제 해결 - redis에 액세스 토큰에 대한 로그아웃한 기록 존재한 것이 원인 - Date 타입 객체의 toInstant() 호출시 UTC 기준으로 반환된 것이 문제 * #386 feat: 차트 조회시 포트폴리오 디테일 프로퍼티 추가 - 포트폴리오 디테일 프로퍼티에는 등록번호(id), 증권사(securitiesFirm), 이름(name)이 포함되어 있습니다. - 추가 이유 : 프론트 모바일 환경에서 차트 조회시 포트폴리오에 대한 디테일 정보가 필요하기 때문

* fix: oauth 계정의 프로필 변경 문제 해결 * #386 fix: local bucket 경로 변경 * #384 test: oauth 계정의 프로필 유지 테스트 구현

* #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}

@authorize

* 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: 관심 종목 리소스 소유 검증 애노테이션 추가

@authorize

* 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 목차 정리

yein-lee and others added 30 commits January 18, 2024 16:28

fix: #146 fix price null

468dd33

Merge pull request #147 from fine-ants/fix/price-null

cd1c9a1

[fix] price null

#145 refactor: 인증코드 확인 api 분리

b9d7c2e

Merge pull request #148 from fine-ants/refactor/#145-verifCode

3fa6131

[refactor] 인증코드 확인 api 분리

[feat] 계정 API, 프로필 정보 변경 API 구현 (#151)

e74592f

* #149 feat: 비밀번호 변경 api 구현 * #149 feat: 로그아웃 후 계정 삭제 * #149 feat: 프로필 정보 변경 기능 구현 * #149 fix: controller 메소드 명 변경

#154 fix: 랜덤 닉네임 생성 길이 10자로 변경 (#155)

f37886a

#157 fix: 회원 프로필 변경 에러 해결 (#158)

02f3a93

#157 fix: 컨트롤러에 value 및 consume 설정

e78482f

#157 fix: 회원가입 로깅 필터 추가

c143c7e

#160 fix: 계정 삭제 버그 수정 (#161)

f978723

#162 fix: 비밀번호 변경 문제 해결 (#163)

1f6a6d6

- 데이터베이스에 반영하도록 변경

feat: #165 change watchlist name

94b2948

[fix] 대시보드 서비스 클래스 테스트 코드 문제 (#169)

8dad324

* #167 test: 테스트 코드 실패 수정 - currentPriceManager 목객체를 추가하여 redis를 모킹 처리함 * #121 test: @beforeeach에 배당금 제거 추가

feat: #165 change watchlist name, watchlist had stock api

c04b363

feat: #165 change watchlist name, watchlist had stock api

15d6335

test: #165 controller test

a999c8f

test: #165 service test

96fad2e

Merge pull request #171 from fine-ants/feat/change-watchlist-name

3db2911

[feat] change watchlist name

feat: #173 stock detail get

7a07c93

test: #173 stock detail get controller test

0792996

fix: #175 endpoint naming

a0e7d98

Update src/test/java/codesquad/fineants/spring/api/stock/StockRestCon…

526bef9

…trollerTest.java Co-authored-by: YongHwan Kim <dragonbead95@naver.com>

Update src/main/java/codesquad/fineants/domain/stock/Stock.java

df23841

Co-authored-by: YongHwan Kim <dragonbead95@naver.com>

Update src/main/java/codesquad/fineants/spring/api/success/code/Stock…

13fe97c

…SuccessCode.java Co-authored-by: YongHwan Kim <dragonbead95@naver.com>

Update src/main/java/codesquad/fineants/spring/api/stock/StockRestCon…

e72d64e

…troller.java Co-authored-by: YongHwan Kim <dragonbead95@naver.com>

refactor: #173 response name

4a84286

yonghwankim-dev added 26 commits June 16, 2024 15:36

#371 fix: 액세스 토큰 갱신 수정 (#378)

b34c616

- 액세스 및 리프레시 토큰 갱신하는 경우에만 리스폰스에 쿠키 설정하도록 변경

[fix] oauth 설정 오타 수정 (#380)

a5fcdec

* feat: 액세스 토큰 갱신 롤백 * #371 fix: oauth 설정 오타 수정

[fix] kakao oauth redirect url 수정 (#381)

d8bfc95

* feat: 액세스 토큰 갱신 롤백 * #371 fix: oauth 설정 오타 수정 * #371 fix: oauth 설정 오타 수정 * #371 fix: oauth 설정 오타 수정

#371 fix: 릴리즈 서버 포트 변경

83eec52

#371 fix: 릴리즈 서버 포트 변경

2db7337

[fix] 중복 Set-Cookie 생성 문제 해결 (#383)

c7c634b

* fix: method source 수정 * #371 fix: 토큰 갱신 문제 해결 - 필터를 대상으로 스프링 빈 등록으로 인하여 자동 등록됨

[fix] 일반 로그인 실패 문제 해결 (#385)

770aecd

* #384 fix: 로그인 실패시 400 응답되도록 수정 * #384 test: 로그인 예외 케이스 실패 해결

#386 fix: cors 설정에서 오타 수정 (#387)

338a891

- PUT 메서드 방식 추가

[test] 테스트 임시 비활성화 (#388)

94b5fd4

* #386 fix: cors 설정에서 오타 수정 - PUT 메서드 방식 추가 * #386 test: 엑세스 토큰 갱신 테스트 2개 임시 비활성화

[fix] oauth 계정의 프로필 변경 유지 문제 해결 (#391)

0b96488

* fix: oauth 계정의 프로필 변경 문제 해결 * #386 fix: local bucket 경로 변경 * #384 test: oauth 계정의 프로필 유지 테스트 구현

[feat] 종목 스크롤 검색 (#396)

8cb5247

* build: QueryDSL 의존성 추가 * feat: getSliceOfStock * test: 종목 검색 테스트 추가 * test: 종목 스크롤 검색 Rest Docs 추가 * docs: 종목 스크롤 검색 Rest Dcos 설명문 추가

fix: JPAQueryFactory 의존성 문제 해결

9245214

fix: 테스트 실패 해결

17af5ba

fix: dev -> production으로 변경

5cbfd0a

fix: secret 갱신

666ecc0

fix: db connection property 오타 수정

a00e4c3

yonghwankim-dev added the bug Something isn't working label Jul 23, 2024

Merge branch 'main' into release

e9630fd

yonghwankim-dev temporarily deployed to aws July 23, 2024 05:52 — with GitHub Actions Inactive

yonghwankim-dev merged commit a68b251 into main Jul 23, 2024
1 check passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[fix] property 오타 수정 #406

[fix] property 오타 수정 #406

yonghwankim-dev commented Jul 23, 2024

[fix] property 오타 수정 #406

[fix] property 오타 수정 #406

Conversation

yonghwankim-dev commented Jul 23, 2024

구현한 것