Merge pull request #154 from calderonsamuel/fix-azure-path

MichelNivard · Nov 29, 2023 · 6e57489 · 6e57489
2 parents 5e8c2cf + c5d36c9
commit 6e57489
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/NEWS.md b/NEWS.md
@@ -10,6 +10,7 @@
 
 - Reverted back to use an R6 class for OpenAI streaming. This doesn't affect how the users interact with the addins.
 - Fixed a bug in retrieval of OpenAI models
+- Fixed a bug in Azure OpenAI request formation.
 
 # gptstudio 0.3.1
 

diff --git a/R/azure-openai.R b/R/azure-openai.R
@@ -5,7 +5,7 @@
 #'
 #' @param prompt a list to use as the prompt for generating
 #'   completions
-#' @param task a character string for the API task. Defaults to the Azure OpenAI
+#' @param task a character string for the API task (e.g. "completions"). Defaults to the Azure OpenAI
 #'   task from environment variables if not specified.
 #' @param base_url a character string for the base url. It defaults to the Azure
 #'   OpenAI endpoint from environment variables if not specified.
@@ -48,7 +48,7 @@ request_base_azure_openai <-
       httr2::req_url_path_append("openai/deployments") %>%
       httr2::req_url_path_append(deployment_name) %>%
       httr2::req_url_path_append(task) %>%
-      httr2::req_url_path_append(api_version) %>%
+      httr2::req_url_query("api-version" = api_version) %>%
       httr2::req_headers("api-key" = token,
                          "Content-Type" = "application/json") %>%
       httr2::req_method("POST")

diff --git a/man/create_completion_azure_openai.Rd b/man/create_completion_azure_openai.Rd