[java-opensaml COMMIT] /trunk/opensaml-soap-api/src/main/java/org/opensaml/soap/client/http/AbstractPipelineHttpSOAPC...

noreply at shibboleth.net noreply at shibboleth.net
Fri May 22 18:52:17 EDT 2015


Author: putmanb
Date: Fri May 22 18:52:17 2015
New Revision: 4277

URL: http://svn.shibboleth.net/view/java-opensaml?rev=4277&view=rev
Log:
Avoid unnecessary variable assignments.

Modified:
    trunk/opensaml-soap-api/src/main/java/org/opensaml/soap/client/http/AbstractPipelineHttpSOAPClient.java

Modified: trunk/opensaml-soap-api/src/main/java/org/opensaml/soap/client/http/AbstractPipelineHttpSOAPClient.java
URL: http://svn.shibboleth.net/view/java-opensaml/trunk/opensaml-soap-api/src/main/java/org/opensaml/soap/client/http/AbstractPipelineHttpSOAPClient.java?rev=4277&r1=4276&r2=4277&view=diff
==============================================================================
--- trunk/opensaml-soap-api/src/main/java/org/opensaml/soap/client/http/AbstractPipelineHttpSOAPClient.java	(original)
+++ trunk/opensaml-soap-api/src/main/java/org/opensaml/soap/client/http/AbstractPipelineHttpSOAPClient.java	Fri May 22 18:52:17 2015
@@ -469,25 +469,24 @@
             }
             
             if (securityParameters != null) {
-                List<String> tlsProtocols = securityParameters.getTLSProtocols();
-                if (tlsProtocols != null) {
-                    context.setAttribute(HttpClientSecurityConstants.CONTEXT_KEY_TLS_PROTOCOLS, tlsProtocols);
+                if (securityParameters.getTLSProtocols() != null) {
+                    context.setAttribute(HttpClientSecurityConstants.CONTEXT_KEY_TLS_PROTOCOLS, 
+                            securityParameters.getTLSProtocols());
                 }
                 
-                List<String> tlsCipherSuites = securityParameters.getTLSCipherSuites();
-                if (tlsCipherSuites != null) {
-                    context.setAttribute(HttpClientSecurityConstants.CONTEXT_KEY_TLS_CIPHER_SUITES, tlsCipherSuites);
+                if (securityParameters.getTLSCipherSuites() != null) {
+                    context.setAttribute(HttpClientSecurityConstants.CONTEXT_KEY_TLS_CIPHER_SUITES, 
+                            securityParameters.getTLSCipherSuites());
                 }
                 
-                X509HostnameVerifier hostnameVerifier = securityParameters.getHostnameVerifier();
-                if (hostnameVerifier != null) {
-                    context.setAttribute(HttpClientSecurityConstants.CONTEXT_KEY_HOSTNAME_VERIFIER, hostnameVerifier);
+                if (securityParameters.getHostnameVerifier() != null) {
+                    context.setAttribute(HttpClientSecurityConstants.CONTEXT_KEY_HOSTNAME_VERIFIER, 
+                            securityParameters.getHostnameVerifier());
                 }
                 
-                X509Credential clientTLSCredential = securityParameters.getClientTLSCredential();
-                if (clientTLSCredential != null) {
+                if (securityParameters.getClientTLSCredential() != null) {
                     context.setAttribute(HttpClientSecurityConstants.CONTEXT_KEY_CLIENT_TLS_CREDENTIAL, 
-                            clientTLSCredential);
+                            securityParameters.getClientTLSCredential());
                 }
             }
         }



More information about the commits mailing list