From fc666727897c9f78fdcddbf4b678e1c0b0c413a9 Mon Sep 17 00:00:00 2001 From: Tomasz Wrobel Date: Tue, 16 Jun 2020 15:36:23 +0200 Subject: Remove unnecessary exceptions Issue-ID: AAF-1152 Signed-off-by: Tomasz Wrobel Change-Id: Ie343cd2e3006f9217db5b8454c060eb8c1d3e9e3 --- .../aaf/certservice/client/api/ExitStatus.java | 4 +-- .../conversion/ArtifactsCreatorProvider.java | 18 +---------- .../CertOutputTypeNotSupportedException.java | 35 ---------------------- .../OutputTypeParameterValidationException.java | 34 --------------------- 4 files changed, 2 insertions(+), 89 deletions(-) delete mode 100644 certServiceClient/src/main/java/org/onap/aaf/certservice/client/certification/exception/CertOutputTypeNotSupportedException.java delete mode 100644 certServiceClient/src/main/java/org/onap/aaf/certservice/client/certification/exception/OutputTypeParameterValidationException.java (limited to 'certServiceClient/src/main/java') diff --git a/certServiceClient/src/main/java/org/onap/aaf/certservice/client/api/ExitStatus.java b/certServiceClient/src/main/java/org/onap/aaf/certservice/client/api/ExitStatus.java index 00057829..78ecc778 100644 --- a/certServiceClient/src/main/java/org/onap/aaf/certservice/client/api/ExitStatus.java +++ b/certServiceClient/src/main/java/org/onap/aaf/certservice/client/api/ExitStatus.java @@ -29,9 +29,7 @@ public enum ExitStatus { HTTP_CLIENT_EXCEPTION(6,"Internal HTTP Client connection problem"), PKCS12_CONVERSION_EXCEPTION(7,"Fail in PKCS12 conversion"), PK_TO_PEM_ENCODING_EXCEPTION(8,"Fail in Private Key to PEM Encoding"), - TLS_CONFIGURATION_EXCEPTION(9, "Invalid TLS configuration"), - OUTPUT_TYPE_PARAMETER_VALIDATION_EXCEPTION(10, "Invalid value of the OUTPUT_TYPE parameter"), - CERT_OUTPUT_TYPE_NOT_SUPPORTED_EXCEPTION(11, "Certificate creation type is not supported"); + TLS_CONFIGURATION_EXCEPTION(9, "Invalid TLS configuration"); private final int value; private final String message; diff --git a/certServiceClient/src/main/java/org/onap/aaf/certservice/client/certification/conversion/ArtifactsCreatorProvider.java b/certServiceClient/src/main/java/org/onap/aaf/certservice/client/certification/conversion/ArtifactsCreatorProvider.java index 6fbf373b..ac1eda1a 100644 --- a/certServiceClient/src/main/java/org/onap/aaf/certservice/client/certification/conversion/ArtifactsCreatorProvider.java +++ b/certServiceClient/src/main/java/org/onap/aaf/certservice/client/certification/conversion/ArtifactsCreatorProvider.java @@ -18,12 +18,6 @@ */ package org.onap.aaf.certservice.client.certification.conversion; -import org.onap.aaf.certservice.client.certification.exception.CertOutputTypeNotSupportedException; -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; - -import java.util.Arrays; - public enum ArtifactsCreatorProvider { P12 { @@ -48,18 +42,8 @@ public enum ArtifactsCreatorProvider { } }; - private static final Logger LOGGER = LoggerFactory.getLogger(ArtifactsCreatorProvider.class); - - public static ArtifactsCreator getCreator(String outputType, String outputPath) - throws CertOutputTypeNotSupportedException { - try { - LOGGER.info("Artifact creation type selected: {}", outputType); + public static ArtifactsCreator getCreator(String outputType, String outputPath) { return valueOf(outputType).create(outputPath); - } catch (IllegalArgumentException e) { - LOGGER.error("Artifact creation type: {} is not supported. Supported types: {}", - outputType, Arrays.toString(values())); - throw new CertOutputTypeNotSupportedException(e); - } } abstract ArtifactsCreator create(String outputPath); diff --git a/certServiceClient/src/main/java/org/onap/aaf/certservice/client/certification/exception/CertOutputTypeNotSupportedException.java b/certServiceClient/src/main/java/org/onap/aaf/certservice/client/certification/exception/CertOutputTypeNotSupportedException.java deleted file mode 100644 index 3c9581ac..00000000 --- a/certServiceClient/src/main/java/org/onap/aaf/certservice/client/certification/exception/CertOutputTypeNotSupportedException.java +++ /dev/null @@ -1,35 +0,0 @@ -/*============LICENSE_START======================================================= - * aaf-certservice-client - * ================================================================================ - * Copyright (C) 2020 Nokia. 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========================================================= - */ - -package org.onap.aaf.certservice.client.certification.exception; - -import org.onap.aaf.certservice.client.api.ExitStatus; -import org.onap.aaf.certservice.client.api.ExitableException; - -public class CertOutputTypeNotSupportedException extends ExitableException { - private static final ExitStatus EXIT_STATUS = ExitStatus.CERT_OUTPUT_TYPE_NOT_SUPPORTED_EXCEPTION; - - public CertOutputTypeNotSupportedException(Throwable e) { - super(e); - } - - public ExitStatus applicationExitStatus() { - return EXIT_STATUS; - } -} diff --git a/certServiceClient/src/main/java/org/onap/aaf/certservice/client/certification/exception/OutputTypeParameterValidationException.java b/certServiceClient/src/main/java/org/onap/aaf/certservice/client/certification/exception/OutputTypeParameterValidationException.java deleted file mode 100644 index 2a80fda1..00000000 --- a/certServiceClient/src/main/java/org/onap/aaf/certservice/client/certification/exception/OutputTypeParameterValidationException.java +++ /dev/null @@ -1,34 +0,0 @@ -/*============LICENSE_START======================================================= - * aaf-certservice-client - * ================================================================================ - * Copyright (C) 2020 Nokia. 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========================================================= - */ -package org.onap.aaf.certservice.client.certification.exception; - -import org.onap.aaf.certservice.client.api.ExitStatus; -import org.onap.aaf.certservice.client.api.ExitableException; - -public class OutputTypeParameterValidationException extends ExitableException { - - public OutputTypeParameterValidationException(String paramValue) { - super(String.format("Invalid value of the OUTPUT_TYPE parameter : '%s'", paramValue)); - } - - @Override - public ExitStatus applicationExitStatus() { - return ExitStatus.OUTPUT_TYPE_PARAMETER_VALIDATION_EXCEPTION; - } -} -- cgit 1.2.3-korg