Skip to content
体验新版
项目
组织
正在加载...
登录
切换导航
打开侧边栏
chenpeng_fei
spring-framework
提交
9c55d22f
S
spring-framework
项目概览
chenpeng_fei
/
spring-framework
与 Fork 源项目一致
从无法访问的项目Fork
通知
2
Star
0
Fork
0
代码
文件
提交
分支
Tags
贡献者
分支图
Diff
Issue
0
列表
看板
标记
里程碑
合并请求
0
Wiki
0
Wiki
分析
仓库
DevOps
项目成员
Pages
S
spring-framework
项目概览
项目概览
详情
发布
仓库
仓库
文件
提交
分支
标签
贡献者
分支图
比较
Issue
0
Issue
0
列表
看板
标记
里程碑
合并请求
0
合并请求
0
Pages
分析
分析
仓库分析
DevOps
Wiki
0
Wiki
成员
成员
收起侧边栏
关闭侧边栏
动态
分支图
创建新Issue
提交
Issue看板
前往新版Gitcode,体验更适合开发者的 AI 搜索 >>
提交
9c55d22f
编写于
12月 19, 2016
作者:
J
Juergen Hoeller
浏览文件
操作
浏览文件
下载
电子邮件补丁
差异文件
MBeanExporter silently ignores null beans
Issue: SPR-15031
上级
e0c43c4f
变更
2
隐藏空白更改
内联
并排
Showing
2 changed file
with
88 addition
and
25 deletion
+88
-25
spring-context/src/main/java/org/springframework/jmx/export/MBeanExporter.java
...in/java/org/springframework/jmx/export/MBeanExporter.java
+15
-11
spring-context/src/test/java/org/springframework/jmx/export/MBeanExporterTests.java
...va/org/springframework/jmx/export/MBeanExporterTests.java
+73
-14
未找到文件。
spring-context/src/main/java/org/springframework/jmx/export/MBeanExporter.java
浏览文件 @
9c55d22f
...
...
@@ -83,7 +83,7 @@ import org.springframework.util.ObjectUtils;
* via the {@link #setListeners(MBeanExporterListener[]) listeners} property, allowing
* application code to be notified of MBean registration and unregistration events.
*
* <p>This exporter is compatible with MBeans a
nd MXBeans on Java 6 and above
.
* <p>This exporter is compatible with MBeans a
s well as MXBeans
.
*
* @author Rob Harrop
* @author Juergen Hoeller
...
...
@@ -466,7 +466,7 @@ public class MBeanExporter extends MBeanRegistrationSupport implements MBeanExpo
objectName
=
JmxUtils
.
appendIdentityToObjectName
(
objectName
,
managedResource
);
}
}
catch
(
Exception
ex
)
{
catch
(
Throwable
ex
)
{
throw
new
MBeanExportException
(
"Unable to generate ObjectName for MBean ["
+
managedResource
+
"]"
,
ex
);
}
registerManagedResource
(
managedResource
,
objectName
);
...
...
@@ -578,7 +578,8 @@ public class MBeanExporter extends MBeanRegistrationSupport implements MBeanExpo
* @param mapValue the value configured for this bean in the beans map;
* may be either the {@code String} name of a bean, or the bean itself
* @param beanKey the key associated with this bean in the beans map
* @return the {@code ObjectName} under which the resource was registered
* @return the {@code ObjectName} under which the resource was registered,
* or {@code null} if the actual resource was {@code null} as well
* @throws MBeanExportException if the export failed
* @see #setBeans
* @see #registerBeanInstance
...
...
@@ -599,12 +600,14 @@ public class MBeanExporter extends MBeanRegistrationSupport implements MBeanExpo
}
else
{
Object
bean
=
this
.
beanFactory
.
getBean
(
beanName
);
ObjectName
objectName
=
registerBeanInstance
(
bean
,
beanKey
);
replaceNotificationListenerBeanNameKeysIfNecessary
(
beanName
,
objectName
);
return
objectName
;
if
(
bean
!=
null
)
{
ObjectName
objectName
=
registerBeanInstance
(
bean
,
beanKey
);
replaceNotificationListenerBeanNameKeysIfNecessary
(
beanName
,
objectName
);
return
objectName
;
}
}
}
else
{
else
if
(
mapValue
!=
null
)
{
// Plain bean instance -> register it directly.
if
(
this
.
beanFactory
!=
null
)
{
Map
<
String
,
?>
beansOfSameType
=
...
...
@@ -621,10 +624,11 @@ public class MBeanExporter extends MBeanRegistrationSupport implements MBeanExpo
return
registerBeanInstance
(
mapValue
,
beanKey
);
}
}
catch
(
Exception
ex
)
{
catch
(
Throwable
ex
)
{
throw
new
UnableToRegisterMBeanException
(
"Unable to register MBean ["
+
mapValue
+
"] with key '"
+
beanKey
+
"'"
,
ex
);
}
return
null
;
}
/**
...
...
@@ -816,7 +820,7 @@ public class MBeanExporter extends MBeanRegistrationSupport implements MBeanExpo
mbean
.
setManagedResource
(
managedResource
,
MR_TYPE_OBJECT_REFERENCE
);
return
mbean
;
}
catch
(
Exception
ex
)
{
catch
(
Throwable
ex
)
{
throw
new
MBeanExportException
(
"Could not create ModelMBean for managed resource ["
+
managedResource
+
"] with key '"
+
beanKey
+
"'"
,
ex
);
}
...
...
@@ -984,7 +988,7 @@ public class MBeanExporter extends MBeanRegistrationSupport implements MBeanExpo
}
}
}
catch
(
Exception
ex
)
{
catch
(
Throwable
ex
)
{
throw
new
MBeanExportException
(
"Unable to register NotificationListener"
,
ex
);
}
}
...
...
@@ -1004,7 +1008,7 @@ public class MBeanExporter extends MBeanRegistrationSupport implements MBeanExpo
this
.
server
.
removeNotificationListener
(
mappedObjectName
,
bean
.
getNotificationListener
(),
bean
.
getNotificationFilter
(),
bean
.
getHandback
());
}
catch
(
Exception
ex
)
{
catch
(
Throwable
ex
)
{
if
(
logger
.
isDebugEnabled
())
{
logger
.
debug
(
"Unable to unregister NotificationListener"
,
ex
);
}
...
...
spring-context/src/test/java/org/springframework/jmx/export/MBeanExporterTests.java
浏览文件 @
9c55d22f
...
...
@@ -39,8 +39,10 @@ import org.junit.Test;
import
org.junit.rules.ExpectedException
;
import
org.springframework.aop.framework.ProxyFactory
;
import
org.springframework.beans.factory.FactoryBean
;
import
org.springframework.beans.factory.support.BeanDefinitionBuilder
;
import
org.springframework.beans.factory.support.DefaultListableBeanFactory
;
import
org.springframework.beans.factory.support.RootBeanDefinition
;
import
org.springframework.context.ConfigurableApplicationContext
;
import
org.springframework.context.support.ClassPathXmlApplicationContext
;
import
org.springframework.jmx.AbstractMBeanServerTests
;
...
...
@@ -68,7 +70,6 @@ import static org.junit.Assert.*;
* @author Sam Brannen
* @author Stephane Nicoll
*/
@SuppressWarnings
(
"deprecation"
)
public
class
MBeanExporterTests
extends
AbstractMBeanServerTests
{
@Rule
...
...
@@ -235,7 +236,6 @@ public class MBeanExporterTests extends AbstractMBeanServerTests {
assertListener
(
listener2
);
}
@Test
public
void
testExportJdkProxy
()
throws
Exception
{
JmxTestBean
bean
=
new
JmxTestBean
();
...
...
@@ -541,10 +541,7 @@ public class MBeanExporterTests extends AbstractMBeanServerTests {
start
(
exporter
);
}
/**
* SPR-2158
*/
@Test
@Test
// SPR-2158
public
void
testMBeanIsNotUnregisteredSpuriouslyIfSomeExternalProcessHasUnregisteredMBean
()
throws
Exception
{
MBeanExporter
exporter
=
new
MBeanExporter
();
exporter
.
setBeans
(
getBeanMap
());
...
...
@@ -561,10 +558,7 @@ public class MBeanExporterTests extends AbstractMBeanServerTests {
listener
.
getUnregistered
().
size
());
}
/**
* SPR-3302
*/
@Test
@Test
// SPR-3302
public
void
testBeanNameCanBeUsedInNotificationListenersMap
()
throws
Exception
{
String
beanName
=
"charlesDexterWard"
;
BeanDefinitionBuilder
testBean
=
BeanDefinitionBuilder
.
rootBeanDefinition
(
JmxTestBean
.
class
);
...
...
@@ -608,10 +602,7 @@ public class MBeanExporterTests extends AbstractMBeanServerTests {
start
(
exporter
);
}
/*
* SPR-3625
*/
@Test
@Test
// SPR-3625
public
void
testMBeanIsUnregisteredForRuntimeExceptionDuringInitialization
()
throws
Exception
{
BeanDefinitionBuilder
builder1
=
BeanDefinitionBuilder
.
rootBeanDefinition
(
Person
.
class
);
BeanDefinitionBuilder
builder2
=
BeanDefinitionBuilder
...
...
@@ -667,6 +658,37 @@ public class MBeanExporterTests extends AbstractMBeanServerTests {
ObjectNameManager
.
getInstance
(
secondBeanName
));
}
@Test
public
void
testRegisterFactoryBean
()
throws
MalformedObjectNameException
{
DefaultListableBeanFactory
factory
=
new
DefaultListableBeanFactory
();
factory
.
registerBeanDefinition
(
"spring:type=FactoryBean"
,
new
RootBeanDefinition
(
ProperSomethingFactoryBean
.
class
));
MBeanExporter
exporter
=
new
MBeanExporter
();
exporter
.
setServer
(
getServer
());
exporter
.
setBeanFactory
(
factory
);
exporter
.
setAutodetectMode
(
MBeanExporter
.
AUTODETECT_ALL
);
start
(
exporter
);
assertIsRegistered
(
"Non-null FactoryBean object registered"
,
ObjectNameManager
.
getInstance
(
"spring:type=FactoryBean"
));
}
@Test
public
void
testIgnoreNullObjectFromFactoryBean
()
throws
MalformedObjectNameException
{
DefaultListableBeanFactory
factory
=
new
DefaultListableBeanFactory
();
factory
.
registerBeanDefinition
(
"spring:type=FactoryBean"
,
new
RootBeanDefinition
(
NullSomethingFactoryBean
.
class
));
MBeanExporter
exporter
=
new
MBeanExporter
();
exporter
.
setServer
(
getServer
());
exporter
.
setBeanFactory
(
factory
);
exporter
.
setAutodetectMode
(
MBeanExporter
.
AUTODETECT_ALL
);
start
(
exporter
);
assertIsNotRegistered
(
"Null FactoryBean object not registered"
,
ObjectNameManager
.
getInstance
(
"spring:type=FactoryBean"
));
}
private
ConfigurableApplicationContext
load
(
String
context
)
{
return
new
ClassPathXmlApplicationContext
(
context
,
getClass
());
}
...
...
@@ -799,4 +821,41 @@ public class MBeanExporterTests extends AbstractMBeanServerTests {
}
}
public
interface
SomethingMBean
{}
public
static
class
Something
implements
SomethingMBean
{}
public
static
class
ProperSomethingFactoryBean
implements
FactoryBean
<
Something
>
{
@Override
public
Something
getObject
()
{
return
new
Something
();
}
@Override
public
Class
<?>
getObjectType
()
{
return
Something
.
class
;
}
@Override
public
boolean
isSingleton
()
{
return
true
;
}
}
public
static
class
NullSomethingFactoryBean
implements
FactoryBean
<
Something
>
{
@Override
public
Something
getObject
()
{
return
null
;
}
@Override
public
Class
<?>
getObjectType
()
{
return
Something
.
class
;
}
@Override
public
boolean
isSingleton
()
{
return
true
;
}
}
}
编辑
预览
Markdown
is supported
0%
请重试
或
添加新附件
.
添加附件
取消
You are about to add
0
people
to the discussion. Proceed with caution.
先完成此消息的编辑!
取消
想要评论请
注册
或
登录