ESDK-3407 Fixed java possible fall-through into case warning.

soranat · soranat · commit 8c74c69ddfa6 · 2020-08-26T11:30:11.000-05:00
diff --git a/Java/Eta/ValueAdd/src/main/java/com/thomsonreuters/upa/valueadd/reactor/RestProxyAuthHandler.java b/Java/Eta/ValueAdd/src/main/java/com/thomsonreuters/upa/valueadd/reactor/RestProxyAuthHandler.java
@@ -150,15 +150,11 @@ private int execute(HttpRequestBase httpRequest, RestConnectOptions connOptions,
 						{
 	   	   					Header header = response.getFirstHeader("Location");
 	   	   					try {
-		   	   	                if( header != null )
+		   	   	                if( header != null && header.getValue() != null)
 		   	   	                {
-		   	   	                    String newHost = header.getValue();
-		   	   	                    if ( newHost != null )
-		   	   	                    { 
-		   	   	                    	httpRequest.setURI(new URI(newHost));
-		   	   	                    	done = false;
-		   	   	                    	break;
-		   	   	                    }
+		   	   	                    httpRequest.setURI(new URI(header.getValue()));
+		   	   	                    done = false;
+		   	   	                    break;
 		   	   	                }
 		   	   	                else
 		   	   	                {

Original file line number	Diff line number	Diff line change
`@@ -150,15 +150,11 @@ private int execute(HttpRequestBase httpRequest, RestConnectOptions connOptions,`
`150`	`150`	`{`
`151`	`151`	`Header header = response.getFirstHeader("Location");`
`152`	`152`	`try {`
`153`		`- if( header != null )`
	`153`	`+ if( header != null && header.getValue() != null)`
`154`	`154`	`{`
`155`		`- String newHost = header.getValue();`
`156`		`- if ( newHost != null )`
`157`		`- {`
`158`		`- httpRequest.setURI(new URI(newHost));`
`159`		`- done = false;`
`160`		`- break;`
`161`		`- }`
	`155`	`+ httpRequest.setURI(new URI(header.getValue()));`
	`156`	`+ done = false;`
	`157`	`+ break;`
`162`	`158`	`}`
`163`	`159`	`else`
`164`	`160`	`{`