Merge pull request #59 from KjellBerlin/update-slack-payload-2

Slack payload request param in x-www-form-urlencoded
KjellBerlin · Aug 9, 2024 · 1eb1e4f · 1eb1e4f
2 parents e1d93a0 + 9b27ba8
commit 1eb1e4f
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/src/main/kotlin/com/carbonara/core/delivery/SlackDeliveryWebhookController.kt b/src/main/kotlin/com/carbonara/core/delivery/SlackDeliveryWebhookController.kt
@@ -3,7 +3,6 @@ package com.carbonara.core.delivery
 import com.carbonara.core.order.OrderService
 import mu.KotlinLogging
 import org.springframework.web.bind.annotation.PostMapping
-import org.springframework.web.bind.annotation.RequestParam
 import org.springframework.web.bind.annotation.RestController
 
 @RestController
@@ -13,14 +12,18 @@ class SlackDeliveryWebhookController(
 
     // Potential dos attack endpoint, introduce rate limiting
 
-    @PostMapping("/slack-delivery-status")
-    suspend fun handleSlackWebhook(@RequestParam("payload") payload: String) {
+    @PostMapping("/slack-delivery-status", "application/x-www-form-urlencoded")
+    suspend fun handleSlackWebhook(payload: SlackWebhookRequestBody) {
         log.info("--Start Slack--")
-        log.info(payload)
+        log.info(payload.payload)
         log.info("--End Slack--")
     }
 
     companion object {
         private val log = KotlinLogging.logger {}
     }
 }
+
+data class SlackWebhookRequestBody(
+    val payload: String
+)