X-Git-Url: https://gerrit.onap.org/r/gitweb?a=blobdiff_plain;ds=sidebyside;f=catalog-be%2Fsrc%2Fmain%2Fjava%2Forg%2Fopenecomp%2Fsdc%2Fbe%2Fcomponents%2Fcsar%2FCsarBusinessLogic.java;h=be3a408bfaa6e13dca467d52bdf1baffabaf2e19;hb=216ea4998de9ae4ee603d7f8f5f6725b218ca710;hp=36c16bed9e43ece1f4f386c525897bc7c2be1ab2;hpb=47317bba3db6c7426a5461aa6c20716bf7db8c96;p=sdc.git diff --git a/catalog-be/src/main/java/org/openecomp/sdc/be/components/csar/CsarBusinessLogic.java b/catalog-be/src/main/java/org/openecomp/sdc/be/components/csar/CsarBusinessLogic.java index 36c16bed9e..be3a408bfa 100644 --- a/catalog-be/src/main/java/org/openecomp/sdc/be/components/csar/CsarBusinessLogic.java +++ b/catalog-be/src/main/java/org/openecomp/sdc/be/components/csar/CsarBusinessLogic.java @@ -98,9 +98,7 @@ public class CsarBusinessLogic extends BaseBusinessLogic { } public void validateCsarBeforeCreate(Service resource, String csarUUID) { - // check if VF with the same Csar UUID or with he same name already - - // exists + // check if VF with the same Csar UUID or with he same name already exists StorageOperationStatus status = toscaOperationFacade.validateCsarUuidUniqueness(csarUUID); log.debug("enter validateCsarBeforeCreate,get status:{}", status); if (status == StorageOperationStatus.ENTITY_ALREADY_EXISTS) { @@ -111,7 +109,7 @@ public class CsarBusinessLogic extends BaseBusinessLogic { } } - public OnboardedCsarInfo getCsarInfo(Resource resource, Resource oldResource, User user, Map payload, String csarUUID) { + public OnboardedCsarInfo getCsarInfo(Resource resource, Resource oldResource, User user, Map payload, String csarUUID, AuditingActionEnum auditingAction) { Map csar = payload; if (csar == null) { final var vendorSoftwareProduct = getCsar(resource, user); @@ -119,7 +117,7 @@ public class CsarBusinessLogic extends BaseBusinessLogic { csar = vendorSoftwareProduct.getFileMap(); } - ImmutablePair toscaYamlCsarStatus = validateAndParseCsar(resource, user, csar, csarUUID).left() + ImmutablePair toscaYamlCsarStatus = validateAndParseCsar(resource, user, csar, csarUUID, auditingAction).left() .on(this::throwComponentException); String checksum = CsarValidationUtils.getToscaYamlChecksum(csar, csarUUID, componentsUtils).left() .on(r -> logAndThrowComponentException(r, "Failed to calculate checksum for casrUUID {} error {} ", csarUUID)); @@ -149,9 +147,9 @@ public class CsarBusinessLogic extends BaseBusinessLogic { } } - public ServiceCsarInfo getCsarInfo(Service service, Service oldResource, User user, Map payload, String csarUUID) { + public ServiceCsarInfo getCsarInfo(Service service, Service oldResource, User user, Map payload, String csarUUID, AuditingActionEnum auditingAction) { Map csar = getCsar(service, user, payload, csarUUID); - ImmutablePair toscaYamlCsarStatus = validateAndParseCsar(service, user, csar, csarUUID).left() + ImmutablePair toscaYamlCsarStatus = validateAndParseCsar(service, user, csar, csarUUID, auditingAction).left() .on(this::throwComponentException); String checksum = CsarValidationUtils.getToscaYamlChecksum(csar, csarUUID, componentsUtils).left() .on(r -> logAndThrowComponentException(r, "Failed to calculate checksum for casrUUID {} error {} ", csarUUID)); @@ -199,7 +197,7 @@ public class CsarBusinessLogic extends BaseBusinessLogic { } private Either, ResponseFormat> validateAndParseCsar(Component component, User user, Map csar, - String csarUUID) { + String csarUUID, AuditingActionEnum auditingAction) { Either validateCsarStatus = CsarValidationUtils.validateCsar(csar, csarUUID, componentsUtils); if (validateCsarStatus.isRight()) { ResponseFormat responseFormat = validateCsarStatus.right().value(); @@ -210,7 +208,7 @@ public class CsarBusinessLogic extends BaseBusinessLogic { } return Either.right(responseFormat); } - Either, ResponseFormat> toscaYamlCsarStatus = CsarValidationUtils.getToscaYaml(csar, csarUUID, componentsUtils); + Either, ResponseFormat> toscaYamlCsarStatus = CsarValidationUtils.getToscaYaml(csar, csarUUID, componentsUtils, auditingAction); if (toscaYamlCsarStatus.isRight()) { ResponseFormat responseFormat = toscaYamlCsarStatus.right().value(); log.debug("Error when try to get csar toscayamlFile with csar ID {}, error: {}", csarUUID, responseFormat); @@ -278,7 +276,7 @@ public class CsarBusinessLogic extends BaseBusinessLogic { } private Either, ResponseFormat> validateAndParseCsar(Service service, User user, Map payload, - String csarUUID) { + String csarUUID, AuditingActionEnum auditingAction) { Map csar = getCsar(service, user, payload, csarUUID); Either validateCsarStatus = CsarValidationUtils.validateCsar(csar, csarUUID, componentsUtils); if (validateCsarStatus.isRight()) { @@ -287,7 +285,7 @@ public class CsarBusinessLogic extends BaseBusinessLogic { BeEcompErrorManager.getInstance().logBeDaoSystemError(CREATING_RESOURCE_FROM_CSAR_FETCHING_CSAR_WITH_ID + csarUUID + FAILED); return Either.right(responseFormat); } - Either, ResponseFormat> toscaYamlCsarStatus = CsarValidationUtils.getToscaYaml(csar, csarUUID, componentsUtils); + Either, ResponseFormat> toscaYamlCsarStatus = CsarValidationUtils.getToscaYaml(csar, csarUUID, componentsUtils, auditingAction); if (toscaYamlCsarStatus.isRight()) { ResponseFormat responseFormat = toscaYamlCsarStatus.right().value(); log.debug("Error when try to get csar toscayamlFile with csar ID {}, error: {}", csarUUID, responseFormat);