diff --git a/nachrichten-manager-interface/pom.xml b/nachrichten-manager-interface/pom.xml
index 2c39de2150dc4b2c2ce0b965d0d89f17f04800a2..16eb6c7529d90d3e1eec1dadfbb422deb7be713f 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.7.0</version>
+ <version>4.11.0-SNAPSHOT</version>
<relativePath />
</parent>
@@ -110,7 +111,9 @@
</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-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 824cb1268407e94ee32b51cee7afca21ad4df37a..e163cbd84939285a66bd8aaefa4674529f1ca876 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,7 +25,5 @@ package de.ozgcloud.nachrichten.postfach;
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 93%
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 5724f8624648d2b5deb69a838aeefc51b8123bc9..5a11033b7a4b4d539b48a711f887d6170328091f 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,22 +26,20 @@ package de.ozgcloud.nachrichten.postfach;
import jakarta.validation.constraints.NotBlank;
import lombok.Builder;
-import lombok.Getter;
@Builder
-@Getter
public class StringBasedIdentifier implements PostfachAddressIdentifier {
@NotBlank
private String postfachId;
@Override
- public boolean isStringBasedIdentifier() {
- return true;
+ public String toString() {
+ return postfachId;
}
@Override
- public String toString() {
+ public String getStringRepresentation() {
return postfachId;
}
}
diff --git a/nachrichten-manager-server/pom.xml b/nachrichten-manager-server/pom.xml
index 7e1cae8b053e99ba0e709825190594b31e321c13..1816f0e1d02cbffedacc45318b8eb7fa8a4a51e8 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.7.0</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>
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 ed617fab860928f65acded652b642a15d8c8dd37..a375ec671fd70e5c5b5661cfb0dd57dcb40551d4 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;
@@ -71,12 +70,6 @@ public class InfoManagerService {
}
String getPostfachId(@NonNull PostfachAddress address) {
- var postfachIdentifier = address.getIdentifier();
-
- if (postfachIdentifier.isStringBasedIdentifier()) {
- return ((StringBasedIdentifier) postfachIdentifier).getPostfachId();
- } else {
- throw new IllegalStateException("Illegal Type of Postfach Identifier: " + postfachIdentifier.getClass().getName());
- }
+ return address.getIdentifier().getStringRepresentation();
}
}
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 c521e7dbdb02f729df1d310d89060fbcbefc5c0f..5a3a0fb073de84613e51faf616d9815ce562b8d4 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 9c33343cfeab08ed4aed145415c38402281909a3..8787cb609a1074f07e2f86011b8cd13b8a827789 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 0a2838ee55171c0dfef8b313f5418be12a124aaa..75ec7eaddce83a064d26ed61e58340cfb5156455 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 55f8368dacd8eae7220957419bee650dec58fb33..2d36d96e2638431d57ecf22fed504b78e4975773 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
@@ -35,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;
@@ -138,13 +137,5 @@ class InfoManagerServiceTest {
assertThat(id).isEqualTo(StringBasedIdentifierTestFactory.STRING_BASED_IDENTIFIER_POSTFACH_ID_VALUE);
}
- @Test
- void shouldThrowExceptionForUnkownIdentifer() {
- var identifier = new PostfachAddressIdentifier() {
- };
- var address = PostfachAddressTestFactory.createBuilder().identifier(identifier).build();
-
- assertThatThrownBy(() -> service.getPostfachId(address)).isInstanceOf(IllegalStateException.class);
- }
}
}
diff --git a/pom.xml b/pom.xml
index 404e3644cbbe52229c955caa615b45b54fbb3504..66157833b02c31bf5bbab4014843984d419f49b7 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.7.0</version>
+ <version>4.11.0-SNAPSHOT</version>
</parent>
<groupId>de.ozgcloud.nachrichten</groupId>