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.

Sign up for GitHub

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

Jump to bottom

fix: 로그인 시 Open Redirect 취약점 수정 #566

Merged
merged 1 commit into from
Jun 4, 2024
Merged

fix: 로그인 시 Open Redirect 취약점 수정 #566

merged 1 commit into from
Jun 4, 2024

Conversation

KimTom89
Copy link
Collaborator

@KimTom89 KimTom89 commented Jun 4, 2024

  • Redirect URL을 검증하는 의존성 주입 함수 추가

PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요. PR을 보내기 전에 모든 항목을 확인해야 합니다.

  • 동일한 업데이트/변경에 대한 다른 Pull Requests가 열려있는지 확인했습니다.
  • 테스트가 성공적으로 수행되었는지 확인했습니다.

PR 유형

어떤 유형의 PR인가요? (해당 항목에 모두 체크해주세요)

  • 버그 수정
  • 새로운 기능
  • UI/UX 개선
  • 문서내용 수정
  • 코드 의미에 영향을 주지 않는 변경사항 (오타, 서식 지정, 변수명 변경 등)
  • 코드 리팩토링 (버그 수정이나 기능 변경 없는 코드 변경)
  • 빌드 관련 변경
  • 테스트 코드 추가
  • 기타 (이유를 설명해주세요.)

변경 사항

로그인 시, Redirect URL을 검증하는 의존성 주입 함수 추가

  • validate_login_url

관련 이슈

기타 정보

@KimTom89
[fix] #557 로그인 시 Open Redirect 취약점 수정
eb52096 
- Redirect URL을 검증하는 의존성 주입 함수 추가
@KimTom89 KimTom89 merged commit 4a9a943 into master Jun 4, 2024
@KimTom89 KimTom89 deleted the fix/open_redirect branch June 4, 2024 03:53
@Letm3through Letm3through mentioned this pull request Jun 9, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@KimTom89