[PR #245] [MERGED] Fix: Initialize request object, correct typo, and rename parameter in Azure OpenAI Pipe #354

Closed
opened 2026-02-15 20:15:13 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/open-webui/pipelines/pull/245
Author: @kmishmael
Created: 9/1/2024
Status: Merged
Merged: 9/2/2024
Merged by: @justinh-rahb

Base: mainHead: main


📝 Commits (1)

  • 0303263 fix: initialize request object, correct typo, and rename parameter

📊 Changes

1 file changed (+4 additions, -2 deletions)

View changed files

📝 examples/pipelines/providers/azure_openai_pipeline.py (+4 -2)

📄 Description

This PR addresses the issues identified in the Azure OpenAI Pipe code and references #244:

  1. Initialization of Request Object
    The requests response object r is now initialized to None to prevent a NameError in the exception handling block if an error occurs before r is defined.

  2. Typo Correction
    The typo in the allowed_params set has been corrected from funcions to functions as referenced in the documentation.

  3. Parameter Renaming:
    The parameter dataSources has been renamed to data_sources in the allowed_params set as also referenced in the documentation.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/open-webui/pipelines/pull/245 **Author:** [@kmishmael](https://github.com/kmishmael) **Created:** 9/1/2024 **Status:** ✅ Merged **Merged:** 9/2/2024 **Merged by:** [@justinh-rahb](https://github.com/justinh-rahb) **Base:** `main` ← **Head:** `main` --- ### 📝 Commits (1) - [`0303263`](https://github.com/open-webui/pipelines/commit/030326319770b47809a76731d4476bd73c685950) fix: initialize request object, correct typo, and rename parameter ### 📊 Changes **1 file changed** (+4 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `examples/pipelines/providers/azure_openai_pipeline.py` (+4 -2) </details> ### 📄 Description This PR addresses the issues identified in the Azure OpenAI Pipe code and references #244: 1. Initialization of Request Object The requests response object `r` is now initialized to `None` to prevent a `NameError` in the exception handling block if an error occurs before `r` is defined. 2. Typo Correction The typo in the `allowed_params` set has been corrected from `funcions` to `functions` as referenced in the documentation. 3. Parameter Renaming: The parameter `dataSources` has been renamed to `data_sources` in the `allowed_params` set as also referenced in the [documentation](https://learn.microsoft.com/en-us/azure/ai-services/openai/reference-preview#request-body-2). --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-15 20:15:13 -05:00
yindo closed this issue 2026-02-15 20:15:13 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: open-webui/pipelines#354