From c4127230e0b27ef094b6ef91a40a73b50385692c Mon Sep 17 00:00:00 2001 From: Felix Reichenbach <felix.reichenbach@mgm-tp.com> Date: Wed, 12 Feb 2025 14:46:14 +0100 Subject: [PATCH 1/6] OZG-4094 update common version to 4.10.0 --- nachrichten-manager-interface/pom.xml | 2 +- nachrichten-manager-server/pom.xml | 2 +- pom.xml | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/nachrichten-manager-interface/pom.xml b/nachrichten-manager-interface/pom.xml index 2c39de2..f608bc8 100644 --- a/nachrichten-manager-interface/pom.xml +++ b/nachrichten-manager-interface/pom.xml @@ -31,7 +31,7 @@ <parent> <groupId>de.ozgcloud.common</groupId> <artifactId>ozgcloud-common-dependencies</artifactId> - <version>4.7.0</version> + <version>4.10.0</version> <relativePath /> </parent> diff --git a/nachrichten-manager-server/pom.xml b/nachrichten-manager-server/pom.xml index 7e1cae8..81bf362 100644 --- a/nachrichten-manager-server/pom.xml +++ b/nachrichten-manager-server/pom.xml @@ -49,7 +49,7 @@ <vorgang-manager.version>2.17.0</vorgang-manager.version> <muk-postfach.version>0.1.0</muk-postfach.version> <api-lib.version>0.16.0</api-lib.version> - <ozgcloud-common.version>4.7.0</ozgcloud-common.version> + <ozgcloud-common.version>4.10.0</ozgcloud-common.version> </properties> <dependencies> diff --git a/pom.xml b/pom.xml index 404e364..9388474 100644 --- a/pom.xml +++ b/pom.xml @@ -30,7 +30,7 @@ <parent> <groupId>de.ozgcloud.common</groupId> <artifactId>ozgcloud-common-parent</artifactId> - <version>4.7.0</version> + <version>4.10.0</version> </parent> <groupId>de.ozgcloud.nachrichten</groupId> -- GitLab From cfbe80303156a92a07344649dd9acd5ad98f2a46 Mon Sep 17 00:00:00 2001 From: Felix Reichenbach <felix.reichenbach@mgm-tp.com> Date: Wed, 12 Feb 2025 16:52:50 +0100 Subject: [PATCH 2/6] OZG-4094 set common version to 4.11.0-SNAPSHOT --- nachrichten-manager-interface/pom.xml | 10 ++++++---- nachrichten-manager-server/pom.xml | 2 +- pom.xml | 6 ++++-- 3 files changed, 11 insertions(+), 7 deletions(-) diff --git a/nachrichten-manager-interface/pom.xml b/nachrichten-manager-interface/pom.xml index f608bc8..140434f 100644 --- a/nachrichten-manager-interface/pom.xml +++ b/nachrichten-manager-interface/pom.xml @@ -24,14 +24,15 @@ unter der Lizenz sind dem Lizenztext zu entnehmen. --> -<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" - xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 https://maven.apache.org/xsd/maven-4.0.0.xsd"> +<project xmlns="http://maven.apache.org/POM/4.0.0" + xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" + xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 https://maven.apache.org/xsd/maven-4.0.0.xsd"> <modelVersion>4.0.0</modelVersion> <parent> <groupId>de.ozgcloud.common</groupId> <artifactId>ozgcloud-common-dependencies</artifactId> - <version>4.10.0</version> + <version>4.11.0-SNAPSHOT</version> <relativePath /> </parent> @@ -110,7 +111,8 @@ </outputTarget> <outputTarget> <type>grpc-java</type> - <pluginArtifact>io.grpc:protoc-gen-grpc-java:${protoc-gen.version}</pluginArtifact> + <pluginArtifact> + io.grpc:protoc-gen-grpc-java:${protoc-gen.version}</pluginArtifact> </outputTarget> </outputTargets> </configuration> diff --git a/nachrichten-manager-server/pom.xml b/nachrichten-manager-server/pom.xml index 81bf362..ef72d81 100644 --- a/nachrichten-manager-server/pom.xml +++ b/nachrichten-manager-server/pom.xml @@ -49,7 +49,7 @@ <vorgang-manager.version>2.17.0</vorgang-manager.version> <muk-postfach.version>0.1.0</muk-postfach.version> <api-lib.version>0.16.0</api-lib.version> - <ozgcloud-common.version>4.10.0</ozgcloud-common.version> + <ozgcloud-common.version>4.11.0-SNAPSHOT</ozgcloud-common.version> </properties> <dependencies> diff --git a/pom.xml b/pom.xml index 9388474..6615783 100644 --- a/pom.xml +++ b/pom.xml @@ -24,13 +24,15 @@ unter der Lizenz sind dem Lizenztext zu entnehmen. --> -<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 https://maven.apache.org/xsd/maven-4.0.0.xsd"> +<project xmlns="http://maven.apache.org/POM/4.0.0" + xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" + xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 https://maven.apache.org/xsd/maven-4.0.0.xsd"> <modelVersion>4.0.0</modelVersion> <parent> <groupId>de.ozgcloud.common</groupId> <artifactId>ozgcloud-common-parent</artifactId> - <version>4.10.0</version> + <version>4.11.0-SNAPSHOT</version> </parent> <groupId>de.ozgcloud.nachrichten</groupId> -- GitLab From 220e5b98705972be2d5be652bc753b2bfe19cb10 Mon Sep 17 00:00:00 2001 From: Felix Reichenbach <felix.reichenbach@mgm-tp.com> Date: Thu, 13 Feb 2025 09:49:44 +0100 Subject: [PATCH 3/6] OZG-4094 remove redundant common-lib dependency --- nachrichten-manager-server/pom.xml | 11 ----------- 1 file changed, 11 deletions(-) diff --git a/nachrichten-manager-server/pom.xml b/nachrichten-manager-server/pom.xml index ef72d81..1816f0e 100644 --- a/nachrichten-manager-server/pom.xml +++ b/nachrichten-manager-server/pom.xml @@ -49,7 +49,6 @@ <vorgang-manager.version>2.17.0</vorgang-manager.version> <muk-postfach.version>0.1.0</muk-postfach.version> <api-lib.version>0.16.0</api-lib.version> - <ozgcloud-common.version>4.11.0-SNAPSHOT</ozgcloud-common.version> </properties> <dependencies> @@ -101,12 +100,6 @@ <version>${api-lib.version}</version> </dependency> - <dependency> - <groupId>de.ozgcloud.common</groupId> - <artifactId>ozgcloud-common-lib</artifactId> - <version>${ozgcloud-common.version}</version> - </dependency> - <dependency> <groupId>org.springframework.boot</groupId> <artifactId>spring-boot-starter-mail</artifactId> @@ -241,10 +234,6 @@ <type>test-jar</type> <scope>test</scope> </dependency> - <dependency> - <groupId>de.ozgcloud.common</groupId> - <artifactId>ozgcloud-common-lib</artifactId> - </dependency> </dependencies> <build> -- GitLab From 9a426af3bcc02f9ded51840b33a38d5bf5d6eee9 Mon Sep 17 00:00:00 2001 From: Felix Reichenbach <felix.reichenbach@mgm-tp.com> Date: Fri, 14 Feb 2025 11:39:18 +0100 Subject: [PATCH 4/6] OZG-4094 add interface method to PostfachAddressIdentifier and move StringBasedIdentifier to interface module --- .../nachrichten/postfach/PostfachAddressIdentifier.java | 2 ++ .../nachrichten/postfach/StringBasedIdentifier.java | 7 +++++-- .../de/ozgcloud/nachrichten/info/InfoManagerService.java | 3 +-- .../nachrichten/postfach/PostfachNachrichtMapper.java | 3 +-- .../postfach/bayernid/BayernIdPostfachNachrichtMapper.java | 3 +-- .../nachrichten/postfach/osi/OsiPostfachMessageMapper.java | 5 ++--- .../ozgcloud/nachrichten/info/InfoManagerServiceTest.java | 7 +++++++ 7 files changed, 19 insertions(+), 11 deletions(-) rename {nachrichten-manager-server => nachrichten-manager-postfach-interface}/src/main/java/de/ozgcloud/nachrichten/postfach/StringBasedIdentifier.java (94%) diff --git a/nachrichten-manager-postfach-interface/src/main/java/de/ozgcloud/nachrichten/postfach/PostfachAddressIdentifier.java b/nachrichten-manager-postfach-interface/src/main/java/de/ozgcloud/nachrichten/postfach/PostfachAddressIdentifier.java index 824cb12..2ead26b 100644 --- a/nachrichten-manager-postfach-interface/src/main/java/de/ozgcloud/nachrichten/postfach/PostfachAddressIdentifier.java +++ b/nachrichten-manager-postfach-interface/src/main/java/de/ozgcloud/nachrichten/postfach/PostfachAddressIdentifier.java @@ -28,4 +28,6 @@ public interface PostfachAddressIdentifier { default boolean isStringBasedIdentifier() { return false; } + + String getStringRepresentation(); } diff --git a/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/postfach/StringBasedIdentifier.java b/nachrichten-manager-postfach-interface/src/main/java/de/ozgcloud/nachrichten/postfach/StringBasedIdentifier.java similarity index 94% rename from nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/postfach/StringBasedIdentifier.java rename to nachrichten-manager-postfach-interface/src/main/java/de/ozgcloud/nachrichten/postfach/StringBasedIdentifier.java index 5724f86..5940c9f 100644 --- a/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/postfach/StringBasedIdentifier.java +++ b/nachrichten-manager-postfach-interface/src/main/java/de/ozgcloud/nachrichten/postfach/StringBasedIdentifier.java @@ -26,10 +26,8 @@ package de.ozgcloud.nachrichten.postfach; import jakarta.validation.constraints.NotBlank; import lombok.Builder; -import lombok.Getter; @Builder -@Getter public class StringBasedIdentifier implements PostfachAddressIdentifier { @NotBlank @@ -44,4 +42,9 @@ public class StringBasedIdentifier implements PostfachAddressIdentifier { public String toString() { return postfachId; } + + @Override + public String getStringRepresentation() { + return postfachId; + } } diff --git a/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/info/InfoManagerService.java b/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/info/InfoManagerService.java index ed617fa..3519085 100644 --- a/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/info/InfoManagerService.java +++ b/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/info/InfoManagerService.java @@ -34,7 +34,6 @@ import de.ozgcloud.nachrichten.postfach.PostfachAddress; import de.ozgcloud.nachrichten.postfach.PostfachException; import de.ozgcloud.nachrichten.postfach.PostfachMessageCode; import de.ozgcloud.nachrichten.postfach.PostfachNachricht; -import de.ozgcloud.nachrichten.postfach.StringBasedIdentifier; import lombok.NonNull; import lombok.RequiredArgsConstructor; import lombok.extern.log4j.Log4j2; @@ -74,7 +73,7 @@ public class InfoManagerService { var postfachIdentifier = address.getIdentifier(); if (postfachIdentifier.isStringBasedIdentifier()) { - return ((StringBasedIdentifier) postfachIdentifier).getPostfachId(); + return postfachIdentifier.getStringRepresentation(); } else { throw new IllegalStateException("Illegal Type of Postfach Identifier: " + postfachIdentifier.getClass().getName()); } diff --git a/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/postfach/PostfachNachrichtMapper.java b/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/postfach/PostfachNachrichtMapper.java index c521e7d..5a3a0fb 100644 --- a/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/postfach/PostfachNachrichtMapper.java +++ b/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/postfach/PostfachNachrichtMapper.java @@ -239,7 +239,6 @@ public abstract class PostfachNachrichtMapper { } private Map<String, Object> buildPostfachAddressIdentifierMap(PostfachAddress postfachAddress) { - var identifier = (StringBasedIdentifier) postfachAddress.getIdentifier(); - return Map.of(PostfachAddress.FIELD_POSTFACH_ID, identifier.getPostfachId()); + return Map.of(PostfachAddress.FIELD_POSTFACH_ID, postfachAddress.getIdentifier().getStringRepresentation()); } } \ No newline at end of file diff --git a/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/postfach/bayernid/BayernIdPostfachNachrichtMapper.java b/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/postfach/bayernid/BayernIdPostfachNachrichtMapper.java index 9c33343..8787cb6 100644 --- a/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/postfach/bayernid/BayernIdPostfachNachrichtMapper.java +++ b/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/postfach/bayernid/BayernIdPostfachNachrichtMapper.java @@ -38,7 +38,6 @@ import org.mapstruct.ReportingPolicy; import de.ozgcloud.common.errorhandling.TechnicalException; import de.ozgcloud.nachrichten.postfach.PostfachAddress; import de.ozgcloud.nachrichten.postfach.PostfachNachricht; -import de.ozgcloud.nachrichten.postfach.StringBasedIdentifier; import de.ozgcloud.nachrichten.postfach.bayernid.proxy.GrpcAbsender; import de.ozgcloud.nachrichten.postfach.bayernid.proxy.GrpcAttachmentMetadata; import de.ozgcloud.nachrichten.postfach.bayernid.proxy.GrpcBayernIdMessageMetadata; @@ -85,7 +84,7 @@ interface BayernIdPostfachNachrichtMapper { BayernIdResponse fromSendBayernIdMessageResponse(GrpcSendBayernIdMessageResponse response); default String toPostkorbId(PostfachAddress postfachAddress) { - return ((StringBasedIdentifier) postfachAddress.getIdentifier()).getPostfachId(); + return (postfachAddress.getIdentifier()).getStringRepresentation(); } @Condition diff --git a/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/postfach/osi/OsiPostfachMessageMapper.java b/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/postfach/osi/OsiPostfachMessageMapper.java index 0a2838e..75ec7ea 100644 --- a/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/postfach/osi/OsiPostfachMessageMapper.java +++ b/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/postfach/osi/OsiPostfachMessageMapper.java @@ -35,6 +35,7 @@ import org.springframework.beans.factory.annotation.Autowired; import de.ozgcloud.nachrichten.postfach.FileId; import de.ozgcloud.nachrichten.postfach.PostfachAddress; +import de.ozgcloud.nachrichten.postfach.PostfachAddressIdentifier; import de.ozgcloud.nachrichten.postfach.PostfachNachricht; import de.ozgcloud.nachrichten.postfach.StringBasedIdentifier; @@ -78,9 +79,7 @@ public abstract class OsiPostfachMessageMapper { String toPostfachId(PostfachNachricht nachricht) { return Optional.ofNullable(nachricht.getPostfachAddress()) .map(PostfachAddress::getIdentifier) - .filter(StringBasedIdentifier.class::isInstance) - .map(StringBasedIdentifier.class::cast) - .map(StringBasedIdentifier::getPostfachId) + .map(PostfachAddressIdentifier::getStringRepresentation) .orElse(StringUtils.EMPTY); } diff --git a/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/info/InfoManagerServiceTest.java b/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/info/InfoManagerServiceTest.java index 55f8368..ccf343a 100644 --- a/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/info/InfoManagerServiceTest.java +++ b/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/info/InfoManagerServiceTest.java @@ -28,6 +28,7 @@ import static org.junit.jupiter.api.Assertions.*; import static org.mockito.ArgumentMatchers.*; import static org.mockito.Mockito.*; +import org.apache.commons.lang3.StringUtils; import org.junit.jupiter.api.Nested; import org.junit.jupiter.api.Test; import org.mockito.InjectMocks; @@ -141,7 +142,13 @@ class InfoManagerServiceTest { @Test void shouldThrowExceptionForUnkownIdentifer() { var identifier = new PostfachAddressIdentifier() { + + @Override + public String getStringRepresentation() { + return StringUtils.EMPTY; + } }; + var address = PostfachAddressTestFactory.createBuilder().identifier(identifier).build(); assertThatThrownBy(() -> service.getPostfachId(address)).isInstanceOf(IllegalStateException.class); -- GitLab From b527dc37635b6ff1142bbd0f585aea5e5b5d2564 Mon Sep 17 00:00:00 2001 From: Felix Reichenbach <felix.reichenbach@mgm-tp.com> Date: Tue, 18 Feb 2025 13:58:12 +0100 Subject: [PATCH 5/6] OZG-4094 fix formatting in pom --- nachrichten-manager-interface/pom.xml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/nachrichten-manager-interface/pom.xml b/nachrichten-manager-interface/pom.xml index 140434f..16eb6c7 100644 --- a/nachrichten-manager-interface/pom.xml +++ b/nachrichten-manager-interface/pom.xml @@ -112,7 +112,8 @@ <outputTarget> <type>grpc-java</type> <pluginArtifact> - io.grpc:protoc-gen-grpc-java:${protoc-gen.version}</pluginArtifact> + io.grpc:protoc-gen-grpc-java:${protoc-gen.version} + </pluginArtifact> </outputTarget> </outputTargets> </configuration> -- GitLab From ecb5e37e0732efa08f6acf34f5f9f767fc0280f3 Mon Sep 17 00:00:00 2001 From: Felix Reichenbach <felix.reichenbach@mgm-tp.com> Date: Tue, 18 Feb 2025 15:47:25 +0100 Subject: [PATCH 6/6] OZG-4094 remove isStringBasedIdentifier from PostfachAddressidentifier --- .../postfach/PostfachAddressIdentifier.java | 4 ---- .../postfach/StringBasedIdentifier.java | 5 ----- .../nachrichten/info/InfoManagerService.java | 8 +------- .../nachrichten/info/InfoManagerServiceTest.java | 16 ---------------- 4 files changed, 1 insertion(+), 32 deletions(-) diff --git a/nachrichten-manager-postfach-interface/src/main/java/de/ozgcloud/nachrichten/postfach/PostfachAddressIdentifier.java b/nachrichten-manager-postfach-interface/src/main/java/de/ozgcloud/nachrichten/postfach/PostfachAddressIdentifier.java index 2ead26b..e163cbd 100644 --- a/nachrichten-manager-postfach-interface/src/main/java/de/ozgcloud/nachrichten/postfach/PostfachAddressIdentifier.java +++ b/nachrichten-manager-postfach-interface/src/main/java/de/ozgcloud/nachrichten/postfach/PostfachAddressIdentifier.java @@ -25,9 +25,5 @@ package de.ozgcloud.nachrichten.postfach; public interface PostfachAddressIdentifier { - default boolean isStringBasedIdentifier() { - return false; - } - String getStringRepresentation(); } diff --git a/nachrichten-manager-postfach-interface/src/main/java/de/ozgcloud/nachrichten/postfach/StringBasedIdentifier.java b/nachrichten-manager-postfach-interface/src/main/java/de/ozgcloud/nachrichten/postfach/StringBasedIdentifier.java index 5940c9f..5a11033 100644 --- a/nachrichten-manager-postfach-interface/src/main/java/de/ozgcloud/nachrichten/postfach/StringBasedIdentifier.java +++ b/nachrichten-manager-postfach-interface/src/main/java/de/ozgcloud/nachrichten/postfach/StringBasedIdentifier.java @@ -33,11 +33,6 @@ public class StringBasedIdentifier implements PostfachAddressIdentifier { @NotBlank private String postfachId; - @Override - public boolean isStringBasedIdentifier() { - return true; - } - @Override public String toString() { return postfachId; diff --git a/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/info/InfoManagerService.java b/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/info/InfoManagerService.java index 3519085..a375ec6 100644 --- a/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/info/InfoManagerService.java +++ b/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/info/InfoManagerService.java @@ -70,12 +70,6 @@ public class InfoManagerService { } String getPostfachId(@NonNull PostfachAddress address) { - var postfachIdentifier = address.getIdentifier(); - - if (postfachIdentifier.isStringBasedIdentifier()) { - return postfachIdentifier.getStringRepresentation(); - } else { - throw new IllegalStateException("Illegal Type of Postfach Identifier: " + postfachIdentifier.getClass().getName()); - } + return address.getIdentifier().getStringRepresentation(); } } diff --git a/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/info/InfoManagerServiceTest.java b/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/info/InfoManagerServiceTest.java index ccf343a..2d36d96 100644 --- a/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/info/InfoManagerServiceTest.java +++ b/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/info/InfoManagerServiceTest.java @@ -28,7 +28,6 @@ import static org.junit.jupiter.api.Assertions.*; import static org.mockito.ArgumentMatchers.*; import static org.mockito.Mockito.*; -import org.apache.commons.lang3.StringUtils; import org.junit.jupiter.api.Nested; import org.junit.jupiter.api.Test; import org.mockito.InjectMocks; @@ -36,7 +35,6 @@ import org.mockito.Mock; import org.mockito.Spy; import de.ozgcloud.nachrichten.NachrichtenManagerProperties; -import de.ozgcloud.nachrichten.postfach.PostfachAddressIdentifier; import de.ozgcloud.nachrichten.postfach.PostfachAddressTestFactory; import de.ozgcloud.nachrichten.postfach.PostfachException; import de.ozgcloud.nachrichten.postfach.PostfachMessageCode; @@ -139,19 +137,5 @@ class InfoManagerServiceTest { assertThat(id).isEqualTo(StringBasedIdentifierTestFactory.STRING_BASED_IDENTIFIER_POSTFACH_ID_VALUE); } - @Test - void shouldThrowExceptionForUnkownIdentifer() { - var identifier = new PostfachAddressIdentifier() { - - @Override - public String getStringRepresentation() { - return StringUtils.EMPTY; - } - }; - - var address = PostfachAddressTestFactory.createBuilder().identifier(identifier).build(); - - assertThatThrownBy(() -> service.getPostfachId(address)).isInstanceOf(IllegalStateException.class); - } } } -- GitLab