diff options
author | Isaac Manuel Raj <isaac.manuelraj@huawei.com> | 2019-04-12 11:49:00 +0530 |
---|---|---|
committer | Isaac Manuel Raj <isaac.manuelraj@huawei.com> | 2019-04-12 11:49:00 +0530 |
commit | 3309730ba1c81602bc17baee30484fa8b2a06200 (patch) | |
tree | c37751bc8b1f69096258724c58bcde42b91abed3 /openstack-client-connectors/http-connector | |
parent | d7d05d563996d179e3af89362f5400ab1a1318c9 (diff) |
Code Enhancement (Sonar Fixes)
Sonar Issues Fixes
Issue-ID: SO-1490
Change-Id: Ie016d80897d2cc53003cfb4aba5b9a2b5da9bf4d
Signed-off-by: Isaac Manuel Raj <isaac.manuelraj@huawei.com>
Diffstat (limited to 'openstack-client-connectors/http-connector')
2 files changed, 45 insertions, 67 deletions
diff --git a/openstack-client-connectors/http-connector/src/main/java/com/woorea/openstack/connector/HttpClientConnector.java b/openstack-client-connectors/http-connector/src/main/java/com/woorea/openstack/connector/HttpClientConnector.java index 5a78b30..9b2f2e2 100644 --- a/openstack-client-connectors/http-connector/src/main/java/com/woorea/openstack/connector/HttpClientConnector.java +++ b/openstack-client-connectors/http-connector/src/main/java/com/woorea/openstack/connector/HttpClientConnector.java @@ -63,9 +63,9 @@ import com.woorea.openstack.base.client.OpenStackResponseException; public class HttpClientConnector implements OpenStackClientConnector { - public static ObjectMapper DEFAULT_MAPPER; - public static ObjectMapper WRAPPED_MAPPER; - + public static final ObjectMapper DEFAULT_MAPPER; + public static final ObjectMapper WRAPPED_MAPPER; + private static Logger LOGGER = LoggerFactory.getLogger(HttpClientConnector.class); static { @@ -85,19 +85,19 @@ public class HttpClientConnector implements OpenStackClientConnector { WRAPPED_MAPPER.enable(DeserializationFeature.ACCEPT_SINGLE_VALUE_AS_ARRAY); WRAPPED_MAPPER.disable(DeserializationFeature.FAIL_ON_UNKNOWN_PROPERTIES); } - + protected static <T> ObjectMapper getObjectMapper (Class<T> type) { return type.getAnnotation(JsonRootName.class) == null ? DEFAULT_MAPPER : WRAPPED_MAPPER; } @Override public <T> OpenStackResponse request(OpenStackRequest<T> request) { - - CloseableHttpClient httpClient = null; //HttpClients.createDefault(); + + CloseableHttpClient httpClient = null; httpClient = HttpClients.custom().setRedirectStrategy(new HttpClientRedirectStrategy()).build(); URI uri = null; - + // Build the URI with query params try { URIBuilder uriBuilder = new URIBuilder(request.endpoint() + request.path()); @@ -107,7 +107,7 @@ public class HttpClientConnector implements OpenStackClientConnector { uriBuilder.setParameter(entry.getKey(), String.valueOf(o)); } } - + uri = uriBuilder.build(); } catch (URISyntaxException e) { throw new HttpClientException (e); @@ -116,7 +116,7 @@ public class HttpClientConnector implements OpenStackClientConnector { HttpEntity entity = null; if (request.entity() != null) { // Flatten the entity to a Json string - + try { // Get appropriate mapper, based on existence of a root element in Entity class ObjectMapper mapper = getObjectMapper (request.entity().getEntity().getClass()); @@ -132,35 +132,35 @@ public class HttpClientConnector implements OpenStackClientConnector { throw new HttpClientException ("Json IO error on request entity", e); } } - + // Determine the HttpRequest class based on the method HttpUriRequest httpRequest; - + switch (request.method()) { - case POST: - HttpPost post = new HttpPost(uri); - post.setEntity (entity); - httpRequest = post; - break; - - case GET: - httpRequest = new HttpGet(uri); - break; - - case PUT: - HttpPut put = new HttpPut(uri); - put.setEntity (entity); - httpRequest = put; - break; - - case DELETE: - httpRequest = new HttpDelete(uri); - break; - - default: - throw new HttpClientException ("Unrecognized HTTP Method: " + request.method()); + case POST: + HttpPost post = new HttpPost(uri); + post.setEntity (entity); + httpRequest = post; + break; + + case GET: + httpRequest = new HttpGet(uri); + break; + + case PUT: + HttpPut put = new HttpPut(uri); + put.setEntity (entity); + httpRequest = put; + break; + + case DELETE: + httpRequest = new HttpDelete(uri); + break; + + default: + throw new HttpClientException ("Unrecognized HTTP Method: " + request.method()); } - + for (Entry<String, List<Object>> h : request.headers().entrySet()) { StringBuilder sb = new StringBuilder(); for (Object v : h.getValue()) { @@ -170,25 +170,25 @@ public class HttpClientConnector implements OpenStackClientConnector { } LOGGER.debug ("Sending HTTP request: " + httpRequest.toString()); - + // Get the Response. But don't get the body entity yet, as this response // will be wrapped in an HttpClientResponse. The HttpClientResponse // buffers the body in constructor, so can close the response here. HttpClientResponse httpClientResponse = null; CloseableHttpResponse httpResponse = null; - + // Catch known HttpClient exceptions, and wrap them in OpenStack Client Exceptions // so calling functions can distinguish. Only RuntimeExceptions are allowed. try { httpResponse = httpClient.execute(httpRequest); LOGGER.debug ("Response status: " + httpResponse.getStatusLine().getStatusCode()); - + httpClientResponse = new HttpClientResponse (httpResponse); int status = httpResponse.getStatusLine().getStatusCode(); if (status == HttpStatus.SC_OK || status == HttpStatus.SC_CREATED || - status == HttpStatus.SC_NO_CONTENT || status == HttpStatus.SC_ACCEPTED) + status == HttpStatus.SC_NO_CONTENT || status == HttpStatus.SC_ACCEPTED) { return httpClientResponse; } @@ -219,11 +219,11 @@ public class HttpClientConnector implements OpenStackClientConnector { LOGGER.warn("Unable to close HTTP Response: " + e); } } - + // Get here on an error response (4XX-5XX) throw new OpenStackResponseException(httpResponse.getStatusLine().getReasonPhrase(), - httpResponse.getStatusLine().getStatusCode(), - httpClientResponse); + httpResponse.getStatusLine().getStatusCode(), + httpClientResponse); } -} +}
\ No newline at end of file diff --git a/openstack-client-connectors/http-connector/src/main/java/com/woorea/openstack/connector/HttpClientException.java b/openstack-client-connectors/http-connector/src/main/java/com/woorea/openstack/connector/HttpClientException.java index ffb7379..ecdc1d7 100644 --- a/openstack-client-connectors/http-connector/src/main/java/com/woorea/openstack/connector/HttpClientException.java +++ b/openstack-client-connectors/http-connector/src/main/java/com/woorea/openstack/connector/HttpClientException.java @@ -14,28 +14,6 @@ * ============LICENSE_END=========================================================
*/
-/*
- * ============LICENSE_START==========================================
- * ===================================================================
- * Copyright © 2017 AT&T Intellectual Property. All rights reserved.
- * ===================================================================
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- * ============LICENSE_END============================================
- *
- * ECOMP and OpenECOMP are trademarks
- * and service marks of AT&T Intellectual Property.
- *
- */
package com.woorea.openstack.connector;
@@ -50,12 +28,12 @@ public class HttpClientException extends RuntimeException { public HttpClientException (String s) {
super (s);
}
-
+
public HttpClientException (Exception e) {
super ("Caught nested exception in HttpClient", e);
}
-
+
public HttpClientException (String s, Exception e) {
super (s, e);
}
-}
+}
\ No newline at end of file |