github
/
halo
mirror of https://github.com/halo-dev/halo
1
0
Fork 0
Code Issues Releases Wiki Activity

Fix the problem of not redirecting to corresponding login page after authentication failure (#6896) 

#### What type of PR is this?

/kind bug
/area core
/milestone 2.20.x

#### What this PR does / why we need it:

This PR appends query `method=local` after redirection location in authentication failure handler to redirect to login page with local method.

#### Which issue(s) this PR fixes:

Fixes https://github.com/halo-dev/halo/issues/6894

#### Does this PR introduce a user-facing change?

```release-note
修复非默认登录方式登录失败之后跳转至默认登录方式的问题
```
pull/6898/head
John Niang 2024-10-18 15:29:37 +08:00 committed by GitHub
parent c465bf8c75
commit 697a5e5a4c
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
application/src/main/java/run/halo/app/security/authentication/login/UsernamePasswordHandler.java
View File

@ -65,12 +65,12 @@ public class UsernamePasswordHandler implements ServerAuthenticationSuccessHandl
.filter(ServerWebExchangeMatcher.MatchResult::isMatch)
.switchIfEmpty(Mono.defer(
() -> {
URI location = URI.create("/login?error");
URI location = URI.create("/login?error&method=local");
if (exception instanceof BadCredentialsException) {
location = URI.create("/login?error=invalid-credential");
location = URI.create("/login?error=invalid-credential&method=local");
}
if (exception instanceof TooManyRequestsException) {
location = URI.create("/login?error=rate-limit-exceeded");
location = URI.create("/login?error=rate-limit-exceeded&method=local");
}
return redirectStrategy.sendRedirect(exchange, location);
}).then(Mono.empty())