Use constructor

ngocnhan-tran1996 · ngocnhan-tran1996 · commit d984adb6c92b · 2025-06-20T03:58:09.000+07:00
Signed-off-by: Tran Ngoc Nhan &lt;ngocnhan.tran1996@gmail.com&gt;
diff --git a/config/src/main/java/org/springframework/security/config/authentication/AbstractUserDetailsServiceBeanDefinitionParser.java b/config/src/main/java/org/springframework/security/config/authentication/AbstractUserDetailsServiceBeanDefinitionParser.java
@@ -74,7 +74,7 @@ private String resolveId(Element element, AbstractBeanDefinition definition, Par
 				id = pc.getReaderContext().generateBeanName(definition);
 			}
 			BeanDefinition container = pc.getContainingBeanDefinition();
-			container.setAttribute("userDetailsService", new RuntimeBeanReference(id));
+			container.getConstructorArgumentValues().addIndexedArgumentValue(0, new RuntimeBeanReference(id));
 		}
 		if (StringUtils.hasText(id)) {
 			return id;
diff --git a/config/src/main/java/org/springframework/security/config/authentication/AuthenticationProviderBeanDefinitionParser.java b/config/src/main/java/org/springframework/security/config/authentication/AuthenticationProviderBeanDefinitionParser.java
@@ -64,7 +64,7 @@ public BeanDefinition parse(Element element, ParserContext pc) {
 							+ "elements '" + Elements.USER_SERVICE + "', '" + Elements.JDBC_USER_SERVICE + "' or '"
 							+ Elements.LDAP_USER_SERVICE + "'", element);
 			}
-			authProvider.setAttribute("userDetailsService", new RuntimeBeanReference(ref));
+			authProvider.getConstructorArgumentValues().addIndexedArgumentValue(0, new RuntimeBeanReference(ref));
 		}
 		else {
 			// Use the child elements to create the UserDetailsService

Original file line number	Diff line number	Diff line change
`@@ -74,7 +74,7 @@ private String resolveId(Element element, AbstractBeanDefinition definition, Par`
`74`	`74`	`id = pc.getReaderContext().generateBeanName(definition);`
`75`	`75`	`}`
`76`	`76`	`BeanDefinition container = pc.getContainingBeanDefinition();`
`77`		`- container.setAttribute("userDetailsService", new RuntimeBeanReference(id));`
	`77`	`+ container.getConstructorArgumentValues().addIndexedArgumentValue(0, new RuntimeBeanReference(id));`
`78`	`78`	`}`
`79`	`79`	`if (StringUtils.hasText(id)) {`
`80`	`80`	`return id;`
Original file line number	Diff line number	Diff line change
`@@ -64,7 +64,7 @@ public BeanDefinition parse(Element element, ParserContext pc) {`
`64`	`64`	`+ "elements '" + Elements.USER_SERVICE + "', '" + Elements.JDBC_USER_SERVICE + "' or '"`
`65`	`65`	`+ Elements.LDAP_USER_SERVICE + "'", element);`
`66`	`66`	`}`
`67`		`- authProvider.setAttribute("userDetailsService", new RuntimeBeanReference(ref));`
	`67`	`+ authProvider.getConstructorArgumentValues().addIndexedArgumentValue(0, new RuntimeBeanReference(ref));`
`68`	`68`	`}`
`69`	`69`	`else {`
`70`	`70`	`// Use the child elements to create the UserDetailsService`