Include UsernameNotFoundException in BadCredentialsException

big-cir · big-cir · commit e21f02d7ed73 · 2025-01-31T22:52:11.000+09:00
Closes gh-16496 Signed-off-by: dae won <eodnjs01477@gmail.com>
diff --git a/core/src/main/java/org/springframework/security/authentication/dao/AbstractUserDetailsAuthenticationProvider.java b/core/src/main/java/org/springframework/security/authentication/dao/AbstractUserDetailsAuthenticationProvider.java
@@ -137,8 +137,9 @@ public Authentication authenticate(Authentication authentication) throws Authent
 				if (!this.hideUserNotFoundExceptions) {
 					throw ex;
 				}
-				throw new BadCredentialsException(this.messages
-					.getMessage("AbstractUserDetailsAuthenticationProvider.badCredentials", "Bad credentials"));
+				String message = this.messages.getMessage("AbstractUserDetailsAuthenticationProvider.badCredentials",
+						"Bad credentials");
+				throw new BadCredentialsException(message, ex);
 			}
 			Assert.notNull(user, "retrieveUser returned null - a violation of the interface contract");
 		}

Original file line number	Diff line number	Diff line change
`@@ -137,8 +137,9 @@ public Authentication authenticate(Authentication authentication) throws Authent`
`137`	`137`	`if (!this.hideUserNotFoundExceptions) {`
`138`	`138`	`throw ex;`
`139`	`139`	`}`
`140`		`- throw new BadCredentialsException(this.messages`
`141`		`- .getMessage("AbstractUserDetailsAuthenticationProvider.badCredentials", "Bad credentials"));`
	`140`	`+ String message = this.messages.getMessage("AbstractUserDetailsAuthenticationProvider.badCredentials",`
	`141`	`+ "Bad credentials");`
	`142`	`+ throw new BadCredentialsException(message, ex);`
`142`	`143`	`}`
`143`	`144`	`Assert.notNull(user, "retrieveUser returned null - a violation of the interface contract");`
`144`	`145`	`}`