Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

fix: 로그인 시, '올바르지 않은 URL입니다.' 오류 수정 #591

Merged
merged 1 commit into from
Jun 12, 2024
Merged

fix: 로그인 시, '올바르지 않은 URL입니다.' 오류 수정 #591

merged 1 commit into from
Jun 12, 2024

Conversation

KimTom89
Copy link
Collaborator

@KimTom89 KimTom89 commented Jun 12, 2024 •
edited
Loading

PR Checklist

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

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

PR 유형

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

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

변경 사항

원인

validate_login_url() 함수의 조건문이 정상적인 요청의 url 변수도 예외처리 하는 오류

처리

  • validate_login_url() 함수의 URL 유효성 검사 조건문 수정
  • validate_login_url() > allow_urls 리스트 삭제

관련 이슈

기타 정보

@KimTom89
[fix] 로그인 시, '올바르지 않은 URL입니다.' 오류 수정
a536cd4 
- validate_login_url 함수에서 URL 유효성 검사 수정
@KimTom89 KimTom89 merged commit 724da92 into master Jun 12, 2024
# for free to join this conversation on GitHub. Already have an account? # 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