From 7cd563596da72ec9ebde3fdfc875d55b335e898c Mon Sep 17 00:00:00 2001 From: Ruben Fiszel Date: Fri, 15 Sep 2023 01:06:53 +0200 Subject: [PATCH] align capture endpoint with run endpoint --- backend/windmill-api/src/capture.rs | 9 +++++---- backend/windmill-api/src/jobs.rs | 6 +++--- 2 files changed, 8 insertions(+), 7 deletions(-) diff --git a/backend/windmill-api/src/capture.rs b/backend/windmill-api/src/capture.rs index 2a7669156f..f25d07b4b6 100644 --- a/backend/windmill-api/src/capture.rs +++ b/backend/windmill-api/src/capture.rs @@ -9,7 +9,7 @@ use axum::{ extract::{Extension, Path, Query}, routing::{get, post, put}, - Json, Router, + Router, }; use hyper::{HeaderMap, StatusCode}; use serde::Deserialize; @@ -21,7 +21,7 @@ use windmill_common::{ use crate::{ db::{ApiAuthed, DB}, - jobs::add_include_headers, + jobs::{add_include_headers, add_raw_string, JsonOrForm}, }; const KEEP_LAST: i64 = 8; @@ -95,11 +95,12 @@ pub async fn update_payload( Path((w_id, path)): Path<(String, StripPath)>, Query(run_query): Query, headers: HeaderMap, - Json(args): Json>>, + JsonOrForm(args, raw_string): JsonOrForm, ) -> Result { let mut tx = db.begin().await?; - let args = add_include_headers(&run_query.include_header, headers, args.unwrap_or_default()); + let args = add_raw_string(raw_string, args); + sqlx::query!( " UPDATE capture diff --git a/backend/windmill-api/src/jobs.rs b/backend/windmill-api/src/jobs.rs index e65b1d8759..3cb66a5ffb 100644 --- a/backend/windmill-api/src/jobs.rs +++ b/backend/windmill-api/src/jobs.rs @@ -1480,8 +1480,8 @@ struct PreviewFlow { } pub struct JsonOrForm( - Option>, - Option, + pub Option>, + pub Option, ); #[axum::async_trait] @@ -1594,7 +1594,7 @@ fn decode_payload(t: String) -> anyhow::Result { serde_json::from_slice(vec.as_slice()).context("invalid json") } -fn add_raw_string( +pub fn add_raw_string( raw_string: Option, mut args: serde_json::Map, ) -> serde_json::Map {