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] 최신 sdk 사용에 따른 cloudFrontService 및 S3Service validateURL 로직, 에러핸들링 수정 #57

Merged
merged 4 commits into from
Jan 12, 2024

Conversation

sung-silver
Copy link
Contributor

@sung-silver sung-silver commented Jan 12, 2024

📝 PR 타입

  • 기능 추가
  • 기능 수정
  • 기능 삭제
  • 버그 수정
  • 의존성, 환경 변수, 빌드 관련 코드 업데이트

📝 반영 브랜치

📝 변경 사항

📝 테스트 결과

  • cloudFront 코드 수정에 따른 동작 확인
스크린샷 2024-01-12 오전 10 04 30
  • validateURL 에러 확인
스크린샷 2024-01-12 오전 10 26 11

📝 To Reviewer

  • 각 API 별로 pdf, 이미지를 몇 분 동안 조회 가능하게 할지 ExpiredTime을 다르게 설정하면 좋을 것 같은데 어떠신가요? 현재 로직에서는 일단 UniversityExam의 제한 시간으로 모두 설정해두었습니다!

Copy link
Contributor

@mikekks mikekks left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!!
ExpiredTime 다르게 설정하는게 더 좋을 것 같네요! 좋은 의견 감사합니다 ㅎ


compileOnly group: 'org.springframework.cloud', name: 'spring-cloud-starter-aws', version: '2.2.1.RELEASE'

implementation 'software.amazon.awssdk:cloudfront:2.22.3'
Copy link
Contributor

Choose a reason for hiding this comment

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

보안적으로 취약한지는 생각을 못했네요..! 역시 보안전문가십니다 ㅎㅎ

@sung-silver sung-silver merged commit 6259588 into dev Jan 12, 2024
1 check passed
@sung-silver sung-silver deleted the fix/#56 branch January 12, 2024 04:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FIX] AWS 최신 의존성에 따른 코드 수정 및 AWSService validateURL 함수 수정
2 participants