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

#577 로그인 거부시 거부 사유를 로그에 기록 #578

Merged
merged 2 commits into from
Feb 22, 2025

Conversation

kmc7468
Copy link
Member

@kmc7468 kmc7468 commented Feb 22, 2025

Summary

It closes #577

Extra info

Production 환경에서, SPARCS SSO에서 돌아온 후 로그인을 거부하는 이유가 2가지가 있습니다.

  • 카이스트 구성원이 아닌 경우
  • 콜백으로 돌아오기 전/후의 세션이 다른 경우

위와 같은 경우 로그인 실패 페이지로 리다이렉트를 시키는데, 원인을 로그에 기록하도록 개선하였습니다. 특히, 카이스트 구성원이 아니여서 로그인을 거부하는 경우, UID, SID, 학번, 구성원 타입을 로그에 남기도록 하여 CS에 참고할 수 있도록 하였습니다.

@kmc7468 kmc7468 self-assigned this Feb 22, 2025
@kmc7468 kmc7468 linked an issue Feb 22, 2025 that may be closed by this pull request
1 task
@kmc7468 kmc7468 merged commit f2e47ea into dev Feb 22, 2025
1 check passed
@kmc7468 kmc7468 deleted the 577-logging-login-deny-reason branch February 22, 2025 14:25
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.

로그인 거부시 거부 사유를 로그에 기록
1 participant