JndiObjectFactoryBean explicitly only chooses public interfaces as default proxy interfaces (SPR-5869)

This commit is contained in:
Juergen Hoeller 2010-01-31 14:59:27 +00:00
parent 9c44f9252d
commit 59b8a139d7
1 changed files with 8 additions and 3 deletions

View File

@ -1,5 +1,5 @@
/* /*
* Copyright 2002-2009 the original author or authors. * Copyright 2002-2010 the original author or authors.
* *
* Licensed under the Apache License, Version 2.0 (the "License"); * Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License. * you may not use this file except in compliance with the License.
@ -17,7 +17,7 @@
package org.springframework.jndi; package org.springframework.jndi;
import java.lang.reflect.Method; import java.lang.reflect.Method;
import java.lang.reflect.Modifier;
import javax.naming.Context; import javax.naming.Context;
import javax.naming.NamingException; import javax.naming.NamingException;
@ -295,7 +295,12 @@ public class JndiObjectFactoryBean extends JndiObjectLocator implements FactoryB
throw new IllegalStateException( throw new IllegalStateException(
"Cannot deactivate 'lookupOnStartup' without specifying a 'proxyInterface' or 'expectedType'"); "Cannot deactivate 'lookupOnStartup' without specifying a 'proxyInterface' or 'expectedType'");
} }
proxyFactory.setInterfaces(ClassUtils.getAllInterfacesForClass(targetClass, jof.beanClassLoader)); Class[] ifcs = ClassUtils.getAllInterfacesForClass(targetClass, jof.beanClassLoader);
for (Class ifc : ifcs) {
if (Modifier.isPublic(ifc.getModifiers())) {
proxyFactory.addInterface(ifc);
}
}
} }
if (jof.exposeAccessContext) { if (jof.exposeAccessContext) {
proxyFactory.addAdvice(new JndiContextExposingInterceptor(jof.getJndiTemplate())); proxyFactory.addAdvice(new JndiContextExposingInterceptor(jof.getJndiTemplate()));