From 502bb38770a001fc10b8908ce466903a95211897 Mon Sep 17 00:00:00 2001
From: Ed Lee <edilee@mozilla.com>
Date: Sat, 2 Dec 2023 11:31:53 -0800
Subject: [PATCH] server : fix OpenAI API `stop` field to be optional

(cherry picked from commit Mozilla-Ocho/llamafile@e8c92bcb84ae3bcbf0d617b7ee6a5413bcbd58af)
---
 examples/server/server.cpp | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/examples/server/server.cpp b/examples/server/server.cpp
index 0fd42dcba42cd..911f7bbe1f85a 100644
--- a/examples/server/server.cpp
+++ b/examples/server/server.cpp
@@ -2410,9 +2410,7 @@ json oaicompat_completion_params_parse(
     }
 
     // Handle 'stop' field
-    if (body["stop"].is_null()) {
-        llama_params["stop"] = json::array({});
-    } else if (body["stop"].is_string()) {
+    if (body.contains("stop") && body["stop"].is_string()) {
         llama_params["stop"] = json::array({body["stop"].get<std::string>()});
     } else {
         llama_params["stop"] = json_value(body, "stop", json::array());