Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

Adding HTTP Requests support for Azure OpenAI#64116

Merged
arafatkatze merged 7 commits into
mainfrom
supportHTTPRequestsAzure
Jul 30, 2024
Merged

Adding HTTP Requests support for Azure OpenAI#64116
arafatkatze merged 7 commits into
mainfrom
supportHTTPRequestsAzure

Conversation

@arafatkatze

@arafatkatze arafatkatze commented Jul 28, 2024

Copy link
Copy Markdown
Contributor

This PR adds special support for http requests to azure OpenAI and changes special customer configs to use HTTP instead of HTTPS.

Test plan

Tested this PR locally

Changelog

Comment thread internal/completions/client/azureopenai/openai.go Outdated
@arafatkatze arafatkatze force-pushed the supportHTTPRequestsAzure branch from ffad1c4 to adafbfe Compare July 29, 2024 09:25
Comment thread cmd/customer-2315/main.go
Comment thread internal/rockskip Outdated

@emidoots emidoots left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like the right solution to me. Make sure to get security sign-off.

@arafatkatze arafatkatze requested review from a team July 30, 2024 14:50
Comment thread cmd/customer-4512/main.go Outdated
Comment thread cmd/customer-4512/main.go Outdated
Comment thread cmd/customer-2315/main.go Outdated
Comment thread cmd/customer-2315/main.go Outdated
Comment thread cmd/customer-2315/main.go
arafatkatze and others added 4 commits July 30, 2024 17:24
Co-authored-by: Vincent <evict@users.noreply.github.com>
Co-authored-by: Vincent <evict@users.noreply.github.com>
Co-authored-by: Vincent <evict@users.noreply.github.com>
Co-authored-by: Vincent <evict@users.noreply.github.com>
@arafatkatze arafatkatze merged commit 5ce2eea into main Jul 30, 2024
@arafatkatze arafatkatze deleted the supportHTTPRequestsAzure branch July 30, 2024 15:45
arafatkatze added a commit that referenced this pull request Jul 30, 2024
This PR adds special support for http requests to azure OpenAI and
changes special customer configs to use HTTP instead of HTTPS.

Tested this PR locally
<!-- REQUIRED; info at
https://docs-legacy.sourcegraph.com/dev/background-information/testing_principles
-->

<!-- OPTIONAL; info at
https://www.notion.so/sourcegraph/Writing-a-changelog-entry-dd997f411d524caabf0d8d38a24a878c
-->

---------

Co-authored-by: Vincent <evict@users.noreply.github.com>
nelsonjr referenced this pull request Jul 31, 2024
This is a backport PR to add changes from
https://github.com/sourcegraph/sourcegraph/pull/64116 to v5.5.x to main
to create a release of the frontend.


## Test plan

<!-- REQUIRED; info at
https://docs-legacy.sourcegraph.com/dev/background-information/testing_principles
-->

## Changelog

<!-- OPTIONAL; info at
https://www.notion.so/sourcegraph/Writing-a-changelog-entry-dd997f411d524caabf0d8d38a24a878c
-->

---------

Co-authored-by: Vincent <evict@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants