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

[Backend : Fix] 항공권 url, 스프링 시큐리티 수정 #116

Merged
merged 9 commits into from
Aug 6, 2024

Conversation

Dayon-Hong
Copy link
Collaborator

@Dayon-Hong Dayon-Hong commented Aug 6, 2024

#️⃣연관된 이슈

#115
#107
#40

📋 PR 개요

  • 스카이스캐너 url 생성시 편도 url 오류 해결 -> 날짜정보를 String으로 반환할때 "/"를 아예 붙여서 편도와 왕복 상관없게끔 하여 조건문 필요없이 구분 가능
  • 스케줄러를 이용해 매일 새벽 2시에 환율 정보 업데이트 되도록 수정
  • 기존의 강제주입 어노테이션을 통해 토큰을 확인 하던 문제를 jwt 토큰 생성시 권한을 부여해주고, 시큐리티 필터의 접근 제어 설정을 통해 처리해주어 서버 url 접근시 인증이 필요하도록 수정
  • 토큰 재발급 오류 메시지 수정

✅ PR 유형

  • 새로운 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

✏️ 변경 사항

  • 기존에 컨트롤러나 서비스코드에서 사용하지 않을부분에 대해 변경이 많이 되었습니다.

@Dayon-Hong Dayon-Hong self-assigned this Aug 6, 2024
@Dayon-Hong Dayon-Hong merged commit 7d15f4a into dev_backend Aug 6, 2024
1 check passed
@Dayon-Hong Dayon-Hong deleted the backend/feature/107-currency-update branch August 6, 2024 15:01
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.

1 participant