From c0e347b177a4c0945b406e5684220783f7c31bb2 Mon Sep 17 00:00:00 2001
From: OZGCloud <ozgcloud@mgm-tp.com>
Date: Tue, 3 Dec 2024 08:58:54 +0100
Subject: [PATCH] OZG-7014 do not transmit saml token
Sub task: OZG-7276
---
.../fachstelle/vorgang/VorgangRemoteService.java | 5 ++---
.../vorgang/VorgangRemoteServiceTest.java | 14 --------------
2 files changed, 2 insertions(+), 17 deletions(-)
diff --git a/fachstelle-server/src/main/java/de/ozgcloud/fachstelle/vorgang/VorgangRemoteService.java b/fachstelle-server/src/main/java/de/ozgcloud/fachstelle/vorgang/VorgangRemoteService.java
index b0497f7..daf8636 100644
--- a/fachstelle-server/src/main/java/de/ozgcloud/fachstelle/vorgang/VorgangRemoteService.java
+++ b/fachstelle-server/src/main/java/de/ozgcloud/fachstelle/vorgang/VorgangRemoteService.java
@@ -38,12 +38,11 @@ import org.springframework.web.util.UriTemplate;
@Component
class VorgangRemoteService {
- static final String FIND_VORGANG_URI = "/api/vorgang/{vorgangId}/{samlToken}";
+ static final String FIND_VORGANG_URI = "/api/vorgang/{vorgangId}";
private final RestClient fachstellenProxyClient;
private final FachstellenProxyProperties fachstellenProxyProperties;
private final VorgangMapper vorgangMapper;
- private final CurrentUserService currentUserService;
Vorgang findVorgang(String id, String collaborationManagerAddress) {
var response = fachstellenProxyClient.get().uri(buildFindVorgangUri(id))
@@ -61,7 +60,7 @@ class VorgangRemoteService {
}
String buildFindVorgangUri(String vorgangId) {
- return new UriTemplate(FIND_VORGANG_URI).expand(vorgangId, currentUserService.getSamlToken()).toString();
+ return new UriTemplate(FIND_VORGANG_URI).expand(vorgangId).toString();
}
private Vorgang buildVorgangResponse(CollaborationGrpcFindVorgangResponse response) {
diff --git a/fachstelle-server/src/test/java/de/ozgcloud/fachstelle/vorgang/VorgangRemoteServiceTest.java b/fachstelle-server/src/test/java/de/ozgcloud/fachstelle/vorgang/VorgangRemoteServiceTest.java
index bb2b327..5c0c434 100644
--- a/fachstelle-server/src/test/java/de/ozgcloud/fachstelle/vorgang/VorgangRemoteServiceTest.java
+++ b/fachstelle-server/src/test/java/de/ozgcloud/fachstelle/vorgang/VorgangRemoteServiceTest.java
@@ -63,9 +63,6 @@ class VorgangRemoteServiceTest {
@Mock
private VorgangMapper vorgangMapper;
- @Mock
- private CurrentUserService currentUserService;
-
@Nested
class TestFindVorgang {
@@ -138,22 +135,11 @@ class VorgangRemoteServiceTest {
@Test
void shouldReturnVorgangUri() {
- when(currentUserService.getSamlToken()).thenReturn(UserTestFactory.SAML_TOKEN);
-
final String expectedUri = new UriTemplate(FIND_VORGANG_URI).expand(VorgangTestFactory.ID, UserTestFactory.SAML_TOKEN).toString();
final String uri = vorgangRemoteService.buildFindVorgangUri(VorgangTestFactory.ID);
assertThat(uri).isEqualTo(expectedUri);
}
-
- @Test
- void shouldThrowException() {
- when(currentUserService.getSamlToken()).thenThrow(new SamlTokenNotFoundException());
-
- assertThatExceptionOfType(SamlTokenNotFoundException.class).isThrownBy(
- () -> vorgangRemoteService.buildFindVorgangUri(VorgangTestFactory.ID));
- }
-
}
}
\ No newline at end of file
--
GitLab