From f1c7e0f5b220ab00a2354891a49c9f7e59edadb8 Mon Sep 17 00:00:00 2001 From: Ruben Dijkstra Date: Fri, 27 Jan 2017 13:29:42 +0100 Subject: [PATCH] Check for null on the argument instead of the message See gh-1307 Issue: SPR-15196 --- .../cache/interceptor/AbstractCacheInvoker.java | 2 +- .../org/springframework/jms/core/JmsMessagingTemplate.java | 2 +- .../reactive/socket/adapter/JettyWebSocketHandlerAdapter.java | 4 ++-- .../socket/adapter/StandardWebSocketHandlerAdapter.java | 4 ++-- .../socket/adapter/UndertowWebSocketHandlerAdapter.java | 2 +- .../http/server/reactive/ReactorServerHttpRequest.java | 2 +- .../http/server/reactive/ReactorServerHttpResponse.java | 2 +- .../http/server/reactive/RxNettyServerHttpRequest.java | 2 +- .../http/server/reactive/RxNettyServerHttpResponse.java | 2 +- 9 files changed, 11 insertions(+), 11 deletions(-) diff --git a/spring-context/src/main/java/org/springframework/cache/interceptor/AbstractCacheInvoker.java b/spring-context/src/main/java/org/springframework/cache/interceptor/AbstractCacheInvoker.java index 4d3da40022..6930909953 100644 --- a/spring-context/src/main/java/org/springframework/cache/interceptor/AbstractCacheInvoker.java +++ b/spring-context/src/main/java/org/springframework/cache/interceptor/AbstractCacheInvoker.java @@ -37,7 +37,7 @@ public abstract class AbstractCacheInvoker { } protected AbstractCacheInvoker(CacheErrorHandler errorHandler) { - Assert.notNull("ErrorHandler must not be null"); + Assert.notNull(errorHandler, "ErrorHandler must not be null"); this.errorHandler = errorHandler; } diff --git a/spring-jms/src/main/java/org/springframework/jms/core/JmsMessagingTemplate.java b/spring-jms/src/main/java/org/springframework/jms/core/JmsMessagingTemplate.java index 7a22c041a1..e1051f386b 100644 --- a/spring-jms/src/main/java/org/springframework/jms/core/JmsMessagingTemplate.java +++ b/spring-jms/src/main/java/org/springframework/jms/core/JmsMessagingTemplate.java @@ -73,7 +73,7 @@ public class JmsMessagingTemplate extends AbstractMessagingTemplate * Create a {@code JmsMessagingTemplate} instance with the {@link JmsTemplate} to use. */ public JmsMessagingTemplate(JmsTemplate jmsTemplate) { - Assert.notNull("JmsTemplate must not be null"); + Assert.notNull(jmsTemplate, "JmsTemplate must not be null"); this.jmsTemplate = jmsTemplate; } diff --git a/spring-web-reactive/src/main/java/org/springframework/web/reactive/socket/adapter/JettyWebSocketHandlerAdapter.java b/spring-web-reactive/src/main/java/org/springframework/web/reactive/socket/adapter/JettyWebSocketHandlerAdapter.java index f27c056ceb..76ed0b1de0 100644 --- a/spring-web-reactive/src/main/java/org/springframework/web/reactive/socket/adapter/JettyWebSocketHandlerAdapter.java +++ b/spring-web-reactive/src/main/java/org/springframework/web/reactive/socket/adapter/JettyWebSocketHandlerAdapter.java @@ -62,8 +62,8 @@ public class JettyWebSocketHandlerAdapter { public JettyWebSocketHandlerAdapter(WebSocketHandler handler, Function sessionFactory) { - Assert.notNull("WebSocketHandler is required"); - Assert.notNull("'sessionFactory' is required"); + Assert.notNull(handler, "WebSocketHandler is required"); + Assert.notNull(sessionFactory, "'sessionFactory' is required"); this.delegateHandler = handler; this.sessionFactory = sessionFactory; } diff --git a/spring-web-reactive/src/main/java/org/springframework/web/reactive/socket/adapter/StandardWebSocketHandlerAdapter.java b/spring-web-reactive/src/main/java/org/springframework/web/reactive/socket/adapter/StandardWebSocketHandlerAdapter.java index d0c9aa613d..4993ec6d69 100644 --- a/spring-web-reactive/src/main/java/org/springframework/web/reactive/socket/adapter/StandardWebSocketHandlerAdapter.java +++ b/spring-web-reactive/src/main/java/org/springframework/web/reactive/socket/adapter/StandardWebSocketHandlerAdapter.java @@ -53,8 +53,8 @@ public class StandardWebSocketHandlerAdapter extends Endpoint { public StandardWebSocketHandlerAdapter(WebSocketHandler handler, Function sessionFactory) { - Assert.notNull("WebSocketHandler is required"); - Assert.notNull("'sessionFactory' is required"); + Assert.notNull(handler, "WebSocketHandler is required"); + Assert.notNull(sessionFactory, "'sessionFactory' is required"); this.delegateHandler = handler; this.sessionFactory = sessionFactory; } diff --git a/spring-web-reactive/src/main/java/org/springframework/web/reactive/socket/adapter/UndertowWebSocketHandlerAdapter.java b/spring-web-reactive/src/main/java/org/springframework/web/reactive/socket/adapter/UndertowWebSocketHandlerAdapter.java index 313bc0997d..2d2bb7e759 100644 --- a/spring-web-reactive/src/main/java/org/springframework/web/reactive/socket/adapter/UndertowWebSocketHandlerAdapter.java +++ b/spring-web-reactive/src/main/java/org/springframework/web/reactive/socket/adapter/UndertowWebSocketHandlerAdapter.java @@ -47,7 +47,7 @@ public class UndertowWebSocketHandlerAdapter extends AbstractReceiveListener { public UndertowWebSocketHandlerAdapter(UndertowWebSocketSession session) { - Assert.notNull("UndertowWebSocketSession is required"); + Assert.notNull(session, "UndertowWebSocketSession is required"); this.session = session; } diff --git a/spring-web/src/main/java/org/springframework/http/server/reactive/ReactorServerHttpRequest.java b/spring-web/src/main/java/org/springframework/http/server/reactive/ReactorServerHttpRequest.java index fee98d0af8..9a5937a7cb 100644 --- a/spring-web/src/main/java/org/springframework/http/server/reactive/ReactorServerHttpRequest.java +++ b/spring-web/src/main/java/org/springframework/http/server/reactive/ReactorServerHttpRequest.java @@ -55,7 +55,7 @@ public class ReactorServerHttpRequest extends AbstractServerHttpRequest { } private static URI initUri(HttpServerRequest channel) { - Assert.notNull("'channel' must not be null"); + Assert.notNull(channel, "'channel' must not be null"); InetSocketAddress address = channel.remoteAddress(); return (address == null ? URI.create(channel.uri()) : getBaseUrl(address).resolve(channel.uri())); } diff --git a/spring-web/src/main/java/org/springframework/http/server/reactive/ReactorServerHttpResponse.java b/spring-web/src/main/java/org/springframework/http/server/reactive/ReactorServerHttpResponse.java index baab862abe..38c2cdea1b 100644 --- a/spring-web/src/main/java/org/springframework/http/server/reactive/ReactorServerHttpResponse.java +++ b/spring-web/src/main/java/org/springframework/http/server/reactive/ReactorServerHttpResponse.java @@ -50,7 +50,7 @@ public class ReactorServerHttpResponse extends AbstractServerHttpResponse public ReactorServerHttpResponse(HttpServerResponse response, DataBufferFactory bufferFactory) { super(bufferFactory); - Assert.notNull("'response' must not be null."); + Assert.notNull(response, "'response' must not be null."); this.response = response; } diff --git a/spring-web/src/main/java/org/springframework/http/server/reactive/RxNettyServerHttpRequest.java b/spring-web/src/main/java/org/springframework/http/server/reactive/RxNettyServerHttpRequest.java index b1b38bc0ea..26c9d9f0af 100644 --- a/spring-web/src/main/java/org/springframework/http/server/reactive/RxNettyServerHttpRequest.java +++ b/spring-web/src/main/java/org/springframework/http/server/reactive/RxNettyServerHttpRequest.java @@ -62,7 +62,7 @@ public class RxNettyServerHttpRequest extends AbstractServerHttpRequest { } private static URI initUri(HttpServerRequest request) { - Assert.notNull("'request', request must not be null"); + Assert.notNull(request, "'request' must not be null"); return StringUtils.isEmpty(request.getHostHeader()) ? URI.create(request.getUri()) : getBaseUrl(request).resolve(request.getUri()); } diff --git a/spring-web/src/main/java/org/springframework/http/server/reactive/RxNettyServerHttpResponse.java b/spring-web/src/main/java/org/springframework/http/server/reactive/RxNettyServerHttpResponse.java index d1c54a36ef..809f39c31f 100644 --- a/spring-web/src/main/java/org/springframework/http/server/reactive/RxNettyServerHttpResponse.java +++ b/spring-web/src/main/java/org/springframework/http/server/reactive/RxNettyServerHttpResponse.java @@ -56,7 +56,7 @@ public class RxNettyServerHttpResponse extends AbstractServerHttpResponse { public RxNettyServerHttpResponse(HttpServerResponse response, NettyDataBufferFactory dataBufferFactory) { super(dataBufferFactory); - Assert.notNull("'response', response must not be null."); + Assert.notNull(response, "'response' must not be null."); this.response = response; }