Prevent nested proxies and improve error safety in rebind

- Unwrap freshBean if createBean() returns an AOP proxy to avoid
  double-proxied targets where advice executes twice
- Create fresh instance before destroying old target so a failure
  in createBean()/setTargetSource() does not leave the proxy
  pointing at an already-destroyed target
- On setTargetSource failure, destroy freshBean before rethrowing

Signed-off-by: seonghyeoklee <dltjdgur327@gmail.com>

Author: seonghyeok-lee

spring-cloud-context/src/main/java/org/springframework/cloud/context/properties/ConfigurationPropertiesRebinder.java

@@ -140,12 +140,22 @@ private boolean rebind(String name, ApplicationContext appContext) {
 						|| getNeverRefreshable().contains(name)) {
 					return false; // ignore
 				}
-				appContext.getAutowireCapableBeanFactory().destroyBean(target);
 				if (proxied && bean instanceof Advised advised) {
 					Object freshBean = appContext.getAutowireCapableBeanFactory().createBean(target.getClass());
-					advised.setTargetSource(new org.springframework.aop.target.SingletonTargetSource(freshBean));
+					if (AopUtils.isAopProxy(freshBean)) {
+						freshBean = ProxyUtils.getTargetObject(freshBean);
+					}
+					try {
+						advised.setTargetSource(new org.springframework.aop.target.SingletonTargetSource(freshBean));
+					}
+					catch (Exception ex) {
+						appContext.getAutowireCapableBeanFactory().destroyBean(freshBean);
+						throw ex;
+					}
+					appContext.getAutowireCapableBeanFactory().destroyBean(target);
 				}
 				else {
+					appContext.getAutowireCapableBeanFactory().destroyBean(target);
 					appContext.getAutowireCapableBeanFactory().initializeBean(target, name);
 				}
 				return true;