fix: 🐛 improve header order and curl formatting

microsoft · Nov 17, 2024 · 6aea03b · 6aea03b
1 parent 53001bc
commit 6aea03b
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/packages/core/src/fetch.ts b/packages/core/src/fetch.ts
@@ -163,12 +163,12 @@ export function traceFetchPost(
                         ? "Bearer ***" // Mask Bearer tokens
                         : "***") // Mask other authorization headers
             )
-    const cmd = `curl ${url} \\
+    const cmd = `curl "${url}" \\
+--no-buffer \\
 ${Object.entries(headers)
     .map(([k, v]) => `-H "${k}: ${v}"`)
-    .join("\\\n")} \\
+    .join(" \\\n")} \\
 -d '${JSON.stringify(body, null, 2).replace(/'/g, "'\\''")}'
---no-buffer
 `
     if (trace) trace.detailsFenced(`✉️ fetch`, cmd, "bash")
     else logVerbose(cmd)

diff --git a/packages/core/src/openai.ts b/packages/core/src/openai.ts
@@ -203,8 +203,8 @@ export const OpenAIChatCompletion: ChatCompletionHandler = async (
     trace.dispatchChange()
 
     const fetchHeaders: HeadersInit = {
-        ...getConfigHeaders(cfg),
         "Content-Type": "application/json",
+        ...getConfigHeaders(cfg),
         ...(headers || {}),
     }
     traceFetchPost(trace, url, fetchHeaders as any, postReq)