From: Skip Wonnell Date: Thu, 11 Jan 2018 00:13:30 +0000 (-0600) Subject: Sonar blocker issue resolution X-Git-Tag: v1.3.0~427 X-Git-Url: https://gerrit.onap.org/r/gitweb?p=appc.git;a=commitdiff_plain;h=fd71d3e7b6994868f1b49911ae01bec54c6e15e8 Sonar blocker issue resolution Sonar issue resolutions for ConfigArtifactProcessor.java Resolve this blocker issue: https://sonar.onap.org/issues/search#issues=AWDHkCDV-08if2a6w3sT And other sonar findings: https://sonar.onap.org/issues/search#issues=AWDHkCDV-08if2a6w3sU https://sonar.onap.org/issues/search#issues=AWDHkCDU-08if2a6w3sR https://sonar.onap.org/issues/search#issues=AWDHkCDV-08if2a6w3sS Change-Id: I2c4f2bb38a5d1fc07fd13c5220eac9fa16afe04a Issue-ID: APPC-386 Signed-off-by: Skip Wonnell --- diff --git a/appc-sdc-listener/appc-sdc-listener-bundle/.gitignore b/appc-sdc-listener/appc-sdc-listener-bundle/.gitignore index 09e3bc9b2..805c067f6 100644 --- a/appc-sdc-listener/appc-sdc-listener-bundle/.gitignore +++ b/appc-sdc-listener/appc-sdc-listener-bundle/.gitignore @@ -1,2 +1,3 @@ /bin/ /target/ +/target-ide/ diff --git a/appc-sdc-listener/appc-sdc-listener-bundle/src/main/java/org/onap/appc/sdc/artifacts/impl/ConfigArtifactProcessor.java b/appc-sdc-listener/appc-sdc-listener-bundle/src/main/java/org/onap/appc/sdc/artifacts/impl/ConfigArtifactProcessor.java index b9eedc466..b5f5fd5d4 100644 --- a/appc-sdc-listener/appc-sdc-listener-bundle/src/main/java/org/onap/appc/sdc/artifacts/impl/ConfigArtifactProcessor.java +++ b/appc-sdc-listener/appc-sdc-listener-bundle/src/main/java/org/onap/appc/sdc/artifacts/impl/ConfigArtifactProcessor.java @@ -9,37 +9,35 @@ * 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. - * + * * ECOMP is a trademark and service mark of AT&T Intellectual Property. * ============LICENSE_END========================================================= */ package org.onap.appc.sdc.artifacts.impl; -import org.apache.commons.lang.StringUtils; +import java.net.MalformedURLException; +import java.net.URI; import org.onap.appc.adapter.message.EventSender; +import org.onap.appc.exceptions.APPCException; +import org.onap.appc.sdc.artifacts.object.SDCArtifact; import org.onap.appc.sdc.listener.ProviderOperations; import org.onap.appc.sdc.listener.ProviderResponse; import org.onap.appc.sdc.listener.Util; -import org.onap.appc.exceptions.APPCException; -import com.att.eelf.configuration.EELFLogger; -import com.att.eelf.configuration.EELFManager; -import org.onap.appc.sdc.artifacts.object.SDCArtifact; import org.openecomp.sdc.api.IDistributionClient; import org.openecomp.sdc.api.notification.IArtifactInfo; import org.openecomp.sdc.api.notification.INotificationData; import org.openecomp.sdc.api.notification.IResourceInstance; - -import java.net.MalformedURLException; -import java.net.URI; +import com.att.eelf.configuration.EELFLogger; +import com.att.eelf.configuration.EELFManager; /** * Artifact processor for config artifact type @@ -50,6 +48,7 @@ public class ConfigArtifactProcessor extends AbstractArtifactProcessor { /** * returns an instance of ConfigArtifactProcessor + * * @param client an instance of IDistributionClient * @param eventSender an instance of EventSender * @param notification an instance of INotificationData @@ -57,31 +56,36 @@ public class ConfigArtifactProcessor extends AbstractArtifactProcessor { * @param artifact an instance of IArtifactInfo * @param storeUri an instance of URI */ - public ConfigArtifactProcessor(IDistributionClient client, EventSender eventSender, INotificationData notification, IResourceInstance resource, IArtifactInfo artifact, URI storeUri) { - super(client,eventSender,notification,resource,artifact,storeUri); + public ConfigArtifactProcessor(IDistributionClient client, EventSender eventSender, INotificationData notification, + IResourceInstance resource, IArtifactInfo artifact, URI storeUri) { + super(client, eventSender, notification, resource, artifact, storeUri); } @Override public void processArtifact(SDCArtifact artifact) throws APPCException { - String postData = Util.toSdcStoreDocumentInput(notification, resource, super.artifact, artifact.getArtifactContent()); - logger.debug("ConfigArtifactProcessor::processArtifact::postData="+postData); + String postData = + Util.toSdcStoreDocumentInput(notification, resource, super.artifact, artifact.getArtifactContent()); + logger.debug("ConfigArtifactProcessor::processArtifact::postData=" + postData); try { - ProviderOperations providerOperations = new ProviderOperations(); - if (null != storeUri) - logger.debug("ConfigArtifactProcessor::processArtifact::URI is"+storeUri.toString()); - ProviderResponse result = providerOperations.post(storeUri.toURL(), postData, null); - if (result.getStatus() == 200) { - logger.debug("ConfigArtifactProcessor::processArtifact::post request success!!"); - Util.parseResponse(result.getBody()); - } - else { - logger.debug("ConfigArtifactProcessor::processArtifact()::post request failed!! Returned :" - +result.getStatus()+"-Result body- "+result.getBody()); - throw new APPCException("ConfigArtifactProcessor::processArtifact: Invalid status retrurned from post "+result.getStatus()); + if (null != storeUri) { + logger.debug("ConfigArtifactProcessor::processArtifact::URI is" + storeUri.toString()); + ProviderResponse result = ProviderOperations.post(storeUri.toURL(), postData, null); + if (result.getStatus() == 200) { + logger.debug("ConfigArtifactProcessor::processArtifact::post request success!!"); + Util.parseResponse(result.getBody()); + } else { + logger.debug("ConfigArtifactProcessor::processArtifact()::post request failed!! Returned :" + + result.getStatus() + "-Result body- " + result.getBody()); + throw new APPCException( + "ConfigArtifactProcessor::processArtifact: Invalid status retrurned from post " + + result.getStatus()); + } + } else { + logger.error("ConfigArtifactProcessor::processArtifact, storeURI is null"); } } catch (MalformedURLException | APPCException e) { - logger.error("Error processing artifact : " + this.artifact.toString(),e); - throw new APPCException(e.getMessage(),e); + logger.error("Error processing artifact : " + this.artifact.toString(), e); + throw new APPCException(e.getMessage(), e); } } } diff --git a/appc-sdc-listener/appc-sdc-listener-features/.gitignore b/appc-sdc-listener/appc-sdc-listener-features/.gitignore index 2992ae832..dc376ea6a 100644 --- a/appc-sdc-listener/appc-sdc-listener-features/.gitignore +++ b/appc-sdc-listener/appc-sdc-listener-features/.gitignore @@ -1,3 +1,4 @@ /target/ /bin/ /.settings/ +/target-ide/ diff --git a/appc-sdc-listener/appc-yang-generator/.gitignore b/appc-sdc-listener/appc-yang-generator/.gitignore new file mode 100644 index 000000000..eacf31a67 --- /dev/null +++ b/appc-sdc-listener/appc-yang-generator/.gitignore @@ -0,0 +1 @@ +/target-ide/