[NU-1231] Rewrite process actvity endpoints from akka to tapir (#5389)

TouK · Feb 7, 2024 · f2c2bd9 · f2c2bd9
1 parent c892b34
commit f2c2bd9
Show file tree

Hide file tree

Showing 26 changed files with 1,132 additions and 259 deletions.
diff --git a/designer/client/src/http/HttpService.ts b/designer/client/src/http/HttpService.ts
@@ -341,11 +341,10 @@ class HttpService {
     }
 
     addAttachment(processName: ProcessName, versionId: ProcessVersionId, file: File) {
-        const data = new FormData();
-        data.append("attachment", file);
-
         return api
-            .post(`/processes/${encodeURIComponent(processName)}/${versionId}/activity/attachments`, data)
+            .post(`/processes/${encodeURIComponent(processName)}/${versionId}/activity/attachments`, file, {
+                headers: { "Content-Disposition": `attachment; filename="${file.name}"` },
+            })
             .then(() => this.#addInfo(i18next.t("notification.error.attachmentAdded", "Attachment added")))
             .catch((error) =>
                 this.#addError(i18next.t("notification.error.failedToAddAttachment", "Failed to add attachment"), error, true),

diff --git a/designer/server/src/main/scala/pl/touk/nussknacker/ui/api/ProcessActivityResource.scala b/designer/server/src/main/scala/pl/touk/nussknacker/ui/api/ProcessActivityResource.scala
diff --git a/designer/server/src/main/scala/pl/touk/nussknacker/ui/api/ProcessAttachmentService.scala b/designer/server/src/main/scala/pl/touk/nussknacker/ui/api/ProcessAttachmentService.scala
diff --git a/designer/server/src/main/scala/pl/touk/nussknacker/ui/api/ProcessDirectives.scala b/designer/server/src/main/scala/pl/touk/nussknacker/ui/api/ProcessDirectives.scala
@@ -27,7 +27,7 @@ trait ProcessDirectives extends NuPathMatchers {
   }
 
   def processId(processName: ProcessName): Directive1[ProcessIdWithName] = {
-    onSuccess(processService.getProcessId(processName))
+    onSuccess(processService.getProcessIdUnsafe(processName))
       .map(ProcessIdWithName(_, processName))
       .flatMap(provide)
   }