Feature/add proxy configuration #423
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…h to different LLM providers and configure additional settings like proxy and CA bundle 📝 (package.json): update devDependencies to include required types packages for follow-redirects, and add dependencies for http-proxy-agent and https-proxy-agent ♻️ (config.ts): refactor config.ts to include new configuration options for HTTP proxy, CA bundle, timeout, verify SSL, client certificate, and API version 🚀 (azure.ts): update azure.ts to use the new OpenAI package and pass the HTTP agent to handle proxy settings ♻️ (groq.ts): refactor groq.ts to use OpenAI package instead of the deprecated @azure/openai package ♻️ (openAi.ts): refactor openAi.ts to use OpenAI package instead of the deprecated @azure/openai package, and pass the HTTP agent to handle proxy settings 🚀 (set_missing_default_values.ts): add new migration to set missing default values in global config file ♻️ (engine.ts): refactor engine.ts to include API version in the default config object 🔧 (httpAgent.ts): add new httpAgent utility to handle HTTP settings including proxy, CA bundle, timeout, verify SSL, and client certificate
…it from Docker build ⬆️ (package.json): upgrade version from 3.2.2 to 3.2.3 in package.json ♻️ (httpAgent.ts): refactor code in getHttpAgent function to improve readability and maintainability
di-sukharev
reviewed
Nov 19, 2024
| } | ||
|
|
||
| export class AzureEngine implements AiEngine { | ||
| config: AzureAiEngineConfig; | ||
| client: AzureOpenAIClient; | ||
| client: OpenAI; |
| @@ -1,6 +1,6 @@ | |||
| import migration00 from './00_use_single_api_key_and_url'; | |||
| import migration01 from './01_remove_obsolete_config_keys_from_global_file'; | |||
| import migration02 from './02_set_missing_default_values'; | |||
| import migration03 from './03_set_missing_default_values'; | |||
|
@ByJacob nice job, left few comments, also pls change the base branch to dev (we first merge there) and solve conflicts (there are few). to solve the |
|
@ByJacob pls let me know when you fix conflicts, i will merge this as soon as you are ready |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Pull Request Description 🚀
Overview
This pull request updates the Azure library and introduces new features for enhanced configuration options.
Features
Documentation
Notes
Thank you! 🙏