[java-opensaml COMMIT] in /trunk: opensaml-messaging-api/src/main/java/org/opensaml/messaging/pipeline/httpclient/Htt...

[noreply at shibboleth.net]

Author: putmanb
Date: Fri May  8 17:01:58 2015
New Revision: 4271

URL: http://svn.shibboleth.net/view/java-opensaml?rev=4271&view=rev
Log:
Refactor pipeline SOAP client into 2 classes:
1) an abstract impl, suitable for a Spring lookup-method injection approach
2) a concrete factory-based impl, suitable for a Spring ServiceLocatorFactoryBean approach

Added:
    trunk/opensaml-soap-api/src/main/java/org/opensaml/soap/client/http/AbstractPipelineHttpSOAPClient.java
      - copied, changed from r4270, trunk/opensaml-soap-api/src/main/java/org/opensaml/soap/client/http/PipelineHttpSOAPClient.java
    trunk/opensaml-soap-api/src/main/java/org/opensaml/soap/client/http/PipelineFactoryHttpSOAPClient.java   (with props)
Modified:
    trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/pipeline/httpclient/HttpClientMessagePipelineFactory.java
    trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/pipeline/servlet/HttpServletMessagePipelineFactory.java
    trunk/opensaml-soap-api/src/main/java/org/opensaml/soap/client/http/PipelineHttpSOAPClient.java

Modified: trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/pipeline/httpclient/HttpClientMessagePipelineFactory.java
URL: http://svn.shibboleth.net/view/java-opensaml/trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/pipeline/httpclient/HttpClientMessagePipelineFactory.java?rev=4271&r1=4270&r2=4271&view=diff
==============================================================================
--- trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/pipeline/httpclient/HttpClientMessagePipelineFactory.java	(original)
+++ trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/pipeline/httpclient/HttpClientMessagePipelineFactory.java	Fri May  8 17:01:58 2015
@@ -17,6 +17,7 @@
 
 package org.opensaml.messaging.pipeline.httpclient;
 
+import javax.annotation.Nonnull;
 import javax.annotation.Nullable;
 
 
@@ -33,7 +34,7 @@
      * 
      * @return a new pipeline instance
      */
-    HttpClientMessagePipeline<InboundMessageType, OutboundMessageType> newInstance();
+    @Nonnull HttpClientMessagePipeline<InboundMessageType, OutboundMessageType> newInstance();
     
     /**
      * Return a new instance of {@link HttpClientMessagePipeline}.
@@ -42,6 +43,7 @@
      * 
      * @return a new pipeline instance
      */
-    HttpClientMessagePipeline<InboundMessageType, OutboundMessageType> newInstance(@Nullable final String pipelineName);
+    @Nonnull HttpClientMessagePipeline<InboundMessageType, OutboundMessageType> newInstance(
+            @Nullable final String pipelineName);
 
 }

Modified: trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/pipeline/servlet/HttpServletMessagePipelineFactory.java
URL: http://svn.shibboleth.net/view/java-opensaml/trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/pipeline/servlet/HttpServletMessagePipelineFactory.java?rev=4271&r1=4270&r2=4271&view=diff
==============================================================================
--- trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/pipeline/servlet/HttpServletMessagePipelineFactory.java	(original)
+++ trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/pipeline/servlet/HttpServletMessagePipelineFactory.java	Fri May  8 17:01:58 2015
@@ -17,6 +17,7 @@
 
 package org.opensaml.messaging.pipeline.servlet;
 
+import javax.annotation.Nonnull;
 import javax.annotation.Nullable;
 
 
@@ -33,7 +34,7 @@
      * 
      * @return a new pipeline instance
      */
-    HttpServletMessagePipeline<InboundMessageType, OutboundMessageType> newInstance();
+    @Nonnull HttpServletMessagePipeline<InboundMessageType, OutboundMessageType> newInstance();
 
     /**
      * Return a new instance of {@link HttpServletMessagePipelineFactory}.
@@ -42,6 +43,7 @@
      * 
      * @return a new pipeline instance
      */
-    HttpServletMessagePipeline<InboundMessageType, OutboundMessageType> newInstance(@Nullable final String pipelineName);
+    @Nonnull HttpServletMessagePipeline<InboundMessageType, OutboundMessageType> newInstance(
+            @Nullable final String pipelineName);
 
 }

Copied: trunk/opensaml-soap-api/src/main/java/org/opensaml/soap/client/http/AbstractPipelineHttpSOAPClient.java (from r4270, trunk/opensaml-soap-api/src/main/java/org/opensaml/soap/client/http/PipelineHttpSOAPClient.java)
URL: http://svn.shibboleth.net/view/java-opensaml/trunk/opensaml-soap-api/src/main/java/org/opensaml/soap/client/http/AbstractPipelineHttpSOAPClient.java?p2=trunk/opensaml-soap-api/src/main/java/org/opensaml/soap/client/http/AbstractPipelineHttpSOAPClient.java&p1=trunk/opensaml-soap-api/src/main/java/org/opensaml/soap/client/http/PipelineHttpSOAPClient.java&r1=4270&r2=4271&rev=4271&view=diff
==============================================================================
--- trunk/opensaml-soap-api/src/main/java/org/opensaml/soap/client/http/PipelineHttpSOAPClient.java	(original)

[... 248 lines stripped ...]