-
Notifications
You must be signed in to change notification settings - Fork 4
Open
Description
✍️ 개선 내용 요약
oauth 로그인 실패 시 예외처리 방법을 변경할 지 고민 중
현재 코드: 발생할 수 있는 예외에 대비한 커스텀 에러코드 생성 -> 클라이언트 쪽에 에러를 던진 후 거기서 리다이렉트 시키려고 함
다른 방법: 발생할 수 있는 예외에 대비한 메시지를 url에 쿼리스트링으로 담으면서 로그인 페이지로 리다이렉트 시킨다.
🔍 개선 이유
로그인 실패 시 리다이렉트를 바로 시키는 방법이 더 많이 쓰이는 것 같다.
일반적으로는 SimpleUrlAuthenticationFailureHandler 클래스를 상속받아서 리다이렉트 처리를 하던데 이때 url에 에러메시지를 담아보낸다.
setDefaultFailureUrl("/auth/login?error=true&exception="+errorMessage);
🛠️ 개선 작업 상세 내용
No response
📈 예상 결과
No response
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or request
Type
Projects
Status
Todo