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.

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

Fix: 서비스계층 특정 메서드 readonly 해제 #25

Merged
merged 2 commits into from
Oct 16, 2023

Conversation

nohy6630
Copy link
Member

@nohy6630 nohy6630 commented Oct 16, 2023

close #24

  • 사용자가 소셜로그인을 할때 어떤 권한들을 동의했냐에 따라서 카카오, 네이버 응답 json이 크게 달라지네요. 카카오,네이버서버 응답 json에는 존재하지만 매핑되는 dto 클래스에는 없는 필드가 있을 경우 오류가 발생할까봐 해당 dto 클래스에 @JsonIgnoreProperties(ignoreUnknown = true)를 적용해 주었습니다.
  • 서비스 코드에서 클래스 단위에 Transactional(readonly=true)를 사용하였기 때문에 데이터를 db에 쓰는 모든 메서드의 경우 Transactional 어노테이션을 적용해야 했는데 빠트려서 버그가 있었네요! 해당 내용 수정하였습니다

@nohy6630 nohy6630 linked an issue Oct 16, 2023 that may be closed by this pull request
2 tasks
Copy link
Member

@versatile0010 versatile0010 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

확인했습니다!! 고생하셨습니다 👍

@nohy6630 nohy6630 merged commit d4b4000 into dev Oct 16, 2023
1 check passed
@nohy6630 nohy6630 deleted the fix/login_jsonignore branch October 16, 2023 12:14
nohy6630 added a commit that referenced this pull request Oct 17, 2023
* Feat: CI/CD 환경 구축 (#4)

* chore: set git ignore
- add application.yml
- add application.properties

* chore: set git ignore
- add application.yml
- add application.properties

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): spring security 의존성 추가

* Feat(#2): spring security 기본 설정 추가 및 default login page disable

* Chore(#2): 배포 브랜치 범위 변경
- main 브랜치에 대해서만 배포함

* Feat: 예외는 한 곳에서 처리한다. (#7)

* Chore(#5): spring-validation 의존성 추가

* Feat(#5): 커스텀 예외 NumberOneException 구현

* Feat(#5): CustomExceptionContext 구현

* Feat(#5): ErrorResponse 구현
- 예외처리 시 반환할 base response dto

* Feat(#5): NotFoundException 구현

* Feat(#5): BadRequestException 구현

* Feat(#5): 전역 예외 처리 핸들러 구현

* Feat: Swagger, QueryDSL, JPA 셋업 / 대피소 DB 구축 (#11)

* Chore(#8): swagger, querydsl set up

* docs(#8): swagger config

* Feat(#9): base time entity 구현

* Feat(#9): p6spy 셋업

* Feat(#9): 대피소 엔티티 구현

* Feat(#9): 대피소 엔티티 구현

* Chore(#9): docker-compose.yml to EC2 server

* Chore(#9): docker-compose.yml to EC2 server

* Chore(#9): remove job docker-compose.yml to EC2 server

* Feat: 내 주변 대피소 위치 반환 API 구현 (#15)

* Feat(#14): hello controller for health-check
- 불필요한 api 제거

* Feat(#14): 모든 url 허용
- 테스트 용이며 추후 반드시 바꿔줘야함.

* Feat(#14): 대피소 검색 API 구현

* Feat: 소셜로그인 구현 (#16)

* Feat(#10): RestTemplate 이용한 카카오 계정 정보 받아오기 로직 작성

* Feat(#10): 카카오 계정 이메일 추출

* Feat(#10): 네이버 카카오 인가코드 이용해 사용자 정보 추출

* Feat(#10): jwt 인증 구현

* Feat(#10): Redis 이용한 리프레쉬 토큰 구현

* Chore(#10): docker-compose.yml에 redis 추가

* Feat(#10): swagger 내용 작성

* Feat(#10): 토큰 갱신 api 구현

* Refactor(#10): 코드 리팩토링

* Fix(#10): inner class protected 삭제

* Fix(#10): gitignore 오타 수정

* Feat: shelters api 인가 처리 disable (#20)

* docs: readme 수정

* Chore: 이슈 템플릿 추가

* Feat: CI/CD 환경 구축 (#4)

* chore: set git ignore
- add application.yml
- add application.properties

* chore: set git ignore
- add application.yml
- add application.properties

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): spring security 의존성 추가

* Feat(#2): spring security 기본 설정 추가 및 default login page disable

* Chore(#2): 배포 브랜치 범위 변경
- main 브랜치에 대해서만 배포함

* Feat: 예외는 한 곳에서 처리한다. (#7)

* Chore(#5): spring-validation 의존성 추가

* Feat(#5): 커스텀 예외 NumberOneException 구현

* Feat(#5): CustomExceptionContext 구현

* Feat(#5): ErrorResponse 구현
- 예외처리 시 반환할 base response dto

* Feat(#5): NotFoundException 구현

* Feat(#5): BadRequestException 구현

* Feat(#5): 전역 예외 처리 핸들러 구현

* Feat: Swagger, QueryDSL, JPA 셋업 / 대피소 DB 구축 (#11)

* Chore(#8): swagger, querydsl set up

* docs(#8): swagger config

* Feat(#9): base time entity 구현

* Feat(#9): p6spy 셋업

* Feat(#9): 대피소 엔티티 구현

* Feat(#9): 대피소 엔티티 구현

* Chore(#9): docker-compose.yml to EC2 server

* Chore(#9): docker-compose.yml to EC2 server

* Chore(#9): remove job docker-compose.yml to EC2 server

* Chore(#13): add job docker-compose.yml to EC2 server

* Chore(#13): add job docker-compose.yml to EC2 server

* Chore(#13): add job docker-compose.yml to EC2 server

* Chore(#13): cicd target branch set up

* Feat(#14): hello controller for health-check
- 불필요한 api 제거

* Feat(#14): 모든 url 허용
- 테스트 용이며 추후 반드시 바꿔줘야함.

* Feat(#14): 대피소 검색 API 구현

* Feat(#14): 대피소 API 권한 해제
- 추후 다시 인가 처리 적용할 예정

* Fix(#14): url path 오타 수정

---------

Co-authored-by: nohy6630 <[email protected]>

* Fix: application.yml base64 인코딩 (#22)

* fix(#19): application.yml 출력

* Fix(#19): fix 브랜치에서도 cicd 작동되도록 수정

* Fix(#19): ls 테스트

* Fix(#19): base 인코딩 추가

* Fix: 서비스계층 특정 메서드 readonly 해제 (#25)

* Fix(#24): jsonignore 어노테이션 추가

* Fix(#24): db에 쓰기 작업하는 service 메서드에 transactional 추가

* Feat: swagger 에서 jwt 토큰 인증 가능하도록 변경 (#27)

* Fix(#21): shelters api 에 대한 인증 설정

* Fix(#21): swagger 에서 jwt 토큰 인증을 수행할 수 있도록 변경

* Docs(#21): swagger 문구 추가

* Feat: 토큰 발급 api 응답형식 수정 및 여러 커스텀 예외 추가 (#29)

* Feat(#26): 토큰 발급 api 응답형식 수정 및 여러 커스텀 예외 추가

* Fix(#26): 액세스 토큰 주기 수정

* Fix(#26): 액세스 토큰 에러를 만료와 형식 오류로 나눠서 응답

* Chore(#26): 임시 cicd 배포

* Fix(#26): 리프레쉬 토큰 에러 응답 http status 수정 및 valid 추가

* Feat: shelter 엔드포인트 변경 (#30)

* docs: readme 수정

* Chore: 이슈 템플릿 추가

* Feat: CI/CD 환경 구축 (#4)

* chore: set git ignore
- add application.yml
- add application.properties

* chore: set git ignore
- add application.yml
- add application.properties

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): spring security 의존성 추가

* Feat(#2): spring security 기본 설정 추가 및 default login page disable

* Chore(#2): 배포 브랜치 범위 변경
- main 브랜치에 대해서만 배포함

* Feat: 예외는 한 곳에서 처리한다. (#7)

* Chore(#5): spring-validation 의존성 추가

* Feat(#5): 커스텀 예외 NumberOneException 구현

* Feat(#5): CustomExceptionContext 구현

* Feat(#5): ErrorResponse 구현
- 예외처리 시 반환할 base response dto

* Feat(#5): NotFoundException 구현

* Feat(#5): BadRequestException 구현

* Feat(#5): 전역 예외 처리 핸들러 구현

* Feat: Swagger, QueryDSL, JPA 셋업 / 대피소 DB 구축 (#11)

* Chore(#8): swagger, querydsl set up

* docs(#8): swagger config

* Feat(#9): base time entity 구현

* Feat(#9): p6spy 셋업

* Feat(#9): 대피소 엔티티 구현

* Feat(#9): 대피소 엔티티 구현

* Chore(#9): docker-compose.yml to EC2 server

* Chore(#9): docker-compose.yml to EC2 server

* Chore(#9): remove job docker-compose.yml to EC2 server

* Chore(#13): add job docker-compose.yml to EC2 server

* Chore(#13): add job docker-compose.yml to EC2 server

* Chore(#13): add job docker-compose.yml to EC2 server

* Chore(#13): cicd target branch set up

* Feat(#14): hello controller for health-check
- 불필요한 api 제거

* Feat(#14): 모든 url 허용
- 테스트 용이며 추후 반드시 바꿔줘야함.

* Feat(#14): 대피소 검색 API 구현

* Feat(#14): 대피소 API 권한 해제
- 추후 다시 인가 처리 적용할 예정

* Fix(#14): url path 오타 수정

* Feat(#21): Shelter EP 변경

---------

Co-authored-by: nohy6630 <[email protected]>

---------

Co-authored-by: JaeHyeon Lee <[email protected]>
Co-authored-by: versatile0010 <[email protected]>
versatile0010 added a commit that referenced this pull request Oct 19, 2023
* Feat: CI/CD 환경 구축 (#4)

* chore: set git ignore
- add application.yml
- add application.properties

* chore: set git ignore
- add application.yml
- add application.properties

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): spring security 의존성 추가

* Feat(#2): spring security 기본 설정 추가 및 default login page disable

* Chore(#2): 배포 브랜치 범위 변경
- main 브랜치에 대해서만 배포함

* Feat: 예외는 한 곳에서 처리한다. (#7)

* Chore(#5): spring-validation 의존성 추가

* Feat(#5): 커스텀 예외 NumberOneException 구현

* Feat(#5): CustomExceptionContext 구현

* Feat(#5): ErrorResponse 구현
- 예외처리 시 반환할 base response dto

* Feat(#5): NotFoundException 구현

* Feat(#5): BadRequestException 구현

* Feat(#5): 전역 예외 처리 핸들러 구현

* Feat: Swagger, QueryDSL, JPA 셋업 / 대피소 DB 구축 (#11)

* Chore(#8): swagger, querydsl set up

* docs(#8): swagger config

* Feat(#9): base time entity 구현

* Feat(#9): p6spy 셋업

* Feat(#9): 대피소 엔티티 구현

* Feat(#9): 대피소 엔티티 구현

* Chore(#9): docker-compose.yml to EC2 server

* Chore(#9): docker-compose.yml to EC2 server

* Chore(#9): remove job docker-compose.yml to EC2 server

* Feat: 내 주변 대피소 위치 반환 API 구현 (#15)

* Feat(#14): hello controller for health-check
- 불필요한 api 제거

* Feat(#14): 모든 url 허용
- 테스트 용이며 추후 반드시 바꿔줘야함.

* Feat(#14): 대피소 검색 API 구현

* Feat: 소셜로그인 구현 (#16)

* Feat(#10): RestTemplate 이용한 카카오 계정 정보 받아오기 로직 작성

* Feat(#10): 카카오 계정 이메일 추출

* Feat(#10): 네이버 카카오 인가코드 이용해 사용자 정보 추출

* Feat(#10): jwt 인증 구현

* Feat(#10): Redis 이용한 리프레쉬 토큰 구현

* Chore(#10): docker-compose.yml에 redis 추가

* Feat(#10): swagger 내용 작성

* Feat(#10): 토큰 갱신 api 구현

* Refactor(#10): 코드 리팩토링

* Fix(#10): inner class protected 삭제

* Fix(#10): gitignore 오타 수정

* Feat: shelters api 인가 처리 disable (#20)

* docs: readme 수정

* Chore: 이슈 템플릿 추가

* Feat: CI/CD 환경 구축 (#4)

* chore: set git ignore
- add application.yml
- add application.properties

* chore: set git ignore
- add application.yml
- add application.properties

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): spring security 의존성 추가

* Feat(#2): spring security 기본 설정 추가 및 default login page disable

* Chore(#2): 배포 브랜치 범위 변경
- main 브랜치에 대해서만 배포함

* Feat: 예외는 한 곳에서 처리한다. (#7)

* Chore(#5): spring-validation 의존성 추가

* Feat(#5): 커스텀 예외 NumberOneException 구현

* Feat(#5): CustomExceptionContext 구현

* Feat(#5): ErrorResponse 구현
- 예외처리 시 반환할 base response dto

* Feat(#5): NotFoundException 구현

* Feat(#5): BadRequestException 구현

* Feat(#5): 전역 예외 처리 핸들러 구현

* Feat: Swagger, QueryDSL, JPA 셋업 / 대피소 DB 구축 (#11)

* Chore(#8): swagger, querydsl set up

* docs(#8): swagger config

* Feat(#9): base time entity 구현

* Feat(#9): p6spy 셋업

* Feat(#9): 대피소 엔티티 구현

* Feat(#9): 대피소 엔티티 구현

* Chore(#9): docker-compose.yml to EC2 server

* Chore(#9): docker-compose.yml to EC2 server

* Chore(#9): remove job docker-compose.yml to EC2 server

* Chore(#13): add job docker-compose.yml to EC2 server

* Chore(#13): add job docker-compose.yml to EC2 server

* Chore(#13): add job docker-compose.yml to EC2 server

* Chore(#13): cicd target branch set up

* Feat(#14): hello controller for health-check
- 불필요한 api 제거

* Feat(#14): 모든 url 허용
- 테스트 용이며 추후 반드시 바꿔줘야함.

* Feat(#14): 대피소 검색 API 구현

* Feat(#14): 대피소 API 권한 해제
- 추후 다시 인가 처리 적용할 예정

* Fix(#14): url path 오타 수정

---------

Co-authored-by: nohy6630 <[email protected]>

* Fix: application.yml base64 인코딩 (#22)

* fix(#19): application.yml 출력

* Fix(#19): fix 브랜치에서도 cicd 작동되도록 수정

* Fix(#19): ls 테스트

* Fix(#19): base 인코딩 추가

* Fix: 서비스계층 특정 메서드 readonly 해제 (#25)

* Fix(#24): jsonignore 어노테이션 추가

* Fix(#24): db에 쓰기 작업하는 service 메서드에 transactional 추가

* Feat: swagger 에서 jwt 토큰 인증 가능하도록 변경 (#27)

* Fix(#21): shelters api 에 대한 인증 설정

* Fix(#21): swagger 에서 jwt 토큰 인증을 수행할 수 있도록 변경

* Docs(#21): swagger 문구 추가

* Feat: 토큰 발급 api 응답형식 수정 및 여러 커스텀 예외 추가 (#29)

* Feat(#26): 토큰 발급 api 응답형식 수정 및 여러 커스텀 예외 추가

* Fix(#26): 액세스 토큰 주기 수정

* Fix(#26): 액세스 토큰 에러를 만료와 형식 오류로 나눠서 응답

* Chore(#26): 임시 cicd 배포

* Fix(#26): 리프레쉬 토큰 에러 응답 http status 수정 및 valid 추가

* Feat: shelter 엔드포인트 변경 (#30)

* docs: readme 수정

* Chore: 이슈 템플릿 추가

* Feat: CI/CD 환경 구축 (#4)

* chore: set git ignore
- add application.yml
- add application.properties

* chore: set git ignore
- add application.yml
- add application.properties

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): ci-cd setting

* Chore(#2): spring security 의존성 추가

* Feat(#2): spring security 기본 설정 추가 및 default login page disable

* Chore(#2): 배포 브랜치 범위 변경
- main 브랜치에 대해서만 배포함

* Feat: 예외는 한 곳에서 처리한다. (#7)

* Chore(#5): spring-validation 의존성 추가

* Feat(#5): 커스텀 예외 NumberOneException 구현

* Feat(#5): CustomExceptionContext 구현

* Feat(#5): ErrorResponse 구현
- 예외처리 시 반환할 base response dto

* Feat(#5): NotFoundException 구현

* Feat(#5): BadRequestException 구현

* Feat(#5): 전역 예외 처리 핸들러 구현

* Feat: Swagger, QueryDSL, JPA 셋업 / 대피소 DB 구축 (#11)

* Chore(#8): swagger, querydsl set up

* docs(#8): swagger config

* Feat(#9): base time entity 구현

* Feat(#9): p6spy 셋업

* Feat(#9): 대피소 엔티티 구현

* Feat(#9): 대피소 엔티티 구현

* Chore(#9): docker-compose.yml to EC2 server

* Chore(#9): docker-compose.yml to EC2 server

* Chore(#9): remove job docker-compose.yml to EC2 server

* Chore(#13): add job docker-compose.yml to EC2 server

* Chore(#13): add job docker-compose.yml to EC2 server

* Chore(#13): add job docker-compose.yml to EC2 server

* Chore(#13): cicd target branch set up

* Feat(#14): hello controller for health-check
- 불필요한 api 제거

* Feat(#14): 모든 url 허용
- 테스트 용이며 추후 반드시 바꿔줘야함.

* Feat(#14): 대피소 검색 API 구현

* Feat(#14): 대피소 API 권한 해제
- 추후 다시 인가 처리 적용할 예정

* Fix(#14): url path 오타 수정

* Feat(#21): Shelter EP 변경

---------

Co-authored-by: nohy6630 <[email protected]>

* Fix: logintest 응답 수정 (#34)

* Feat(#32): cloud watch setup (#37)

* Fix(#35): 토큰 갱신시 refresh token도 함께 갱신되도록 수정 (#36)

---------

Co-authored-by: Yeongjin Noh <[email protected]>
Co-authored-by: nohy6630 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Fix]: 카카오 네이버 응답 클래스 jsonignore추가
2 participants