From 8799a27e8d73c5e079d549f6d76abf861d75ee9e Mon Sep 17 00:00:00 2001 From: Christof Rabensteiner Date: Mon, 2 Dec 2019 14:26:42 +0100 Subject: Finalize "Removal of JAXB Class Not Found Fix" - Problem: I fixed CompletableFuture.*async within the DeliveryRequest flow, but forgot to do the same for the DeliveryRequestStatus / DeliveryNotification flow. - Solution: Ensure that all CompletableFuture.*async() invocations have Spring's task executor attached. --- .../java/at/gv/egiz/moazs/backend/MsgResponseBackend.java | 10 +++++++--- .../java/at/gv/egiz/moazs/backend/MsgResponseSinkHub.java | 11 +++++++---- .../java/at/gv/egiz/moazs/backend/SaveResponseToFileSink.java | 9 ++++++--- 3 files changed, 20 insertions(+), 10 deletions(-) (limited to 'src/main/java/at/gv/egiz/moazs/backend') diff --git a/src/main/java/at/gv/egiz/moazs/backend/MsgResponseBackend.java b/src/main/java/at/gv/egiz/moazs/backend/MsgResponseBackend.java index 0da9c92..81398da 100644 --- a/src/main/java/at/gv/egiz/moazs/backend/MsgResponseBackend.java +++ b/src/main/java/at/gv/egiz/moazs/backend/MsgResponseBackend.java @@ -31,6 +31,7 @@ import at.gv.zustellung.app2mzs.xsd.DeliveryRequestType; import org.slf4j.Logger; import org.slf4j.LoggerFactory; import org.springframework.beans.factory.annotation.Autowired; +import org.springframework.core.task.TaskExecutor; import org.springframework.stereotype.Component; import java.util.Map; @@ -64,18 +65,21 @@ public class MsgResponseBackend implements Consumer { private final MsgResponseSinkHub hub; private final Map deliveryRequestConfigs; private final MzsDeliveryRequestValidator validator; + private final TaskExecutor taskExecutor; @Autowired public MsgResponseBackend(DeliveryRepository repository, Consumer signatureVerifier, MsgResponseSinkHub hub, Map deliveryRequestConfigs, - MzsDeliveryRequestValidator validator) { + MzsDeliveryRequestValidator validator, + TaskExecutor taskExecutor) { this.repository = repository; this.signatureVerifier = signatureVerifier; this.hub = hub; this.deliveryRequestConfigs = deliveryRequestConfigs; this.validator = validator; + this.taskExecutor = taskExecutor; } /** @@ -91,8 +95,8 @@ public class MsgResponseBackend implements Consumer { @Override public void accept(String responseID) { log.info("Backend accepts MsgResponse with responseID={}.", responseID); - supplyAsync(() -> verifySignature(responseID)) - .thenAcceptAsync(msgResponse -> applySinks(msgResponse)) + supplyAsync(() -> verifySignature(responseID), taskExecutor) + .thenAcceptAsync(msgResponse -> applySinks(msgResponse), taskExecutor) .exceptionally(ex -> { log.error(ex.getMessage(), ex); return null; diff --git a/src/main/java/at/gv/egiz/moazs/backend/MsgResponseSinkHub.java b/src/main/java/at/gv/egiz/moazs/backend/MsgResponseSinkHub.java index 61df4b2..3927055 100644 --- a/src/main/java/at/gv/egiz/moazs/backend/MsgResponseSinkHub.java +++ b/src/main/java/at/gv/egiz/moazs/backend/MsgResponseSinkHub.java @@ -26,6 +26,7 @@ import at.gv.zustellung.app2mzs.xsd.MsgResponseSinksType; import org.slf4j.Logger; import org.slf4j.LoggerFactory; import org.springframework.beans.factory.annotation.Autowired; +import org.springframework.core.task.TaskExecutor; import org.springframework.stereotype.Component; import static java.util.concurrent.CompletableFuture.supplyAsync; @@ -42,13 +43,15 @@ public class MsgResponseSinkHub { private final SaveResponseToFileSink saveResponseSink; private final LogResponseSink logResponseSink; private final ForwardResponseToServiceSink forwardResponseSink; + private final TaskExecutor taskExecutor; @Autowired public MsgResponseSinkHub(SaveResponseToFileSink saveResponseSink, LogResponseSink logResponseSink, - ForwardResponseToServiceSink forwardResponseSink) { + ForwardResponseToServiceSink forwardResponseSink, TaskExecutor taskExecutor) { this.saveResponseSink = saveResponseSink; this.logResponseSink = logResponseSink; this.forwardResponseSink = forwardResponseSink; + this.taskExecutor = taskExecutor; } /** @@ -60,16 +63,16 @@ public class MsgResponseSinkHub { msgResponse.getRootElementLocalPart(), msgResponse.getAppDeliveryID()); if (sinkParams.getSaveResponseToFile().isActive()) { - supplyAsync(() -> saveResponseSink.save(msgResponse, sinkParams.getSaveResponseToFile().getPath())); + supplyAsync(() -> saveResponseSink.save(msgResponse, sinkParams.getSaveResponseToFile().getPath()), taskExecutor); } if (sinkParams.isLogResponse()) { - supplyAsync(() -> logResponseSink.log(msgResponse)); + supplyAsync(() -> logResponseSink.log(msgResponse), taskExecutor); } if (sinkParams.getForwardResponseToService().isActive()) { supplyAsync(() -> forwardResponseSink.send( - msgResponse, sinkParams.getForwardResponseToService().getAppClient())); + msgResponse, sinkParams.getForwardResponseToService().getAppClient()), taskExecutor); } } diff --git a/src/main/java/at/gv/egiz/moazs/backend/SaveResponseToFileSink.java b/src/main/java/at/gv/egiz/moazs/backend/SaveResponseToFileSink.java index c2652d9..c844511 100644 --- a/src/main/java/at/gv/egiz/moazs/backend/SaveResponseToFileSink.java +++ b/src/main/java/at/gv/egiz/moazs/backend/SaveResponseToFileSink.java @@ -36,6 +36,7 @@ import java.nio.charset.StandardCharsets; import java.text.SimpleDateFormat; import java.util.Date; import java.util.concurrent.CompletableFuture; +import java.util.concurrent.Executor; import static at.gv.egiz.moazs.MoaZSException.moaZSException; import static java.lang.String.format; @@ -56,11 +57,13 @@ public class SaveResponseToFileSink { private final SimpleDateFormat isoFormatter; private final Marshaller msgMarshaller; private final DeliveryRepository repository; + private final Executor taskExecutor; @Autowired - public SaveResponseToFileSink(Marshaller msgMarshaller, DeliveryRepository repository) { + public SaveResponseToFileSink(Marshaller msgMarshaller, DeliveryRepository repository, Executor taskExecutor) { this.msgMarshaller = msgMarshaller; this.repository = repository; + this.taskExecutor = taskExecutor; this.isoFormatter = new SimpleDateFormat("yyyy-MM-dd'T'HH:mm:ss"); } @@ -77,13 +80,13 @@ public class SaveResponseToFileSink { response.getRootElementLocalPart(), response.getAppDeliveryID()); var responsePath = generatePath(rootPath, responseID, "xml"); - var storeResponseToFileSystemFuture = supplyAsync(() -> msgMarshaller.marshallXml(response.getResponseAsJAXBElement())) + var storeResponseToFileSystemFuture = supplyAsync(() -> msgMarshaller.marshallXml(response.getResponseAsJAXBElement()), taskExecutor) .thenApply(responseString -> responseString.getBytes(StandardCharsets.UTF_8)) .thenAccept(responseByteArray -> storeToFile(responsePath, responseByteArray)) .exceptionally(ex -> logException(ex, responseID)); var binaryResponsePath = generatePath(rootPath, responseID, "binary.xml"); - var storeBinaryResponseToFileSystemFuture = supplyAsync(() -> repository.retrieveBinaryResponse(responseID).get()) + var storeBinaryResponseToFileSystemFuture = supplyAsync(() -> repository.retrieveBinaryResponse(responseID).get(), taskExecutor) .thenAccept(binaryResponseByteArray -> storeToFile(binaryResponsePath, binaryResponseByteArray)) .exceptionally(ex -> logException(ex, responseID)); -- cgit v1.2.3