Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: removing bearer from auth header when using azure #3626

Merged
merged 2 commits into from
Dec 24, 2023
Merged

fix: removing bearer from auth header when using azure #3626

merged 2 commits into from
Dec 24, 2023

Conversation

fredliang44
Copy link
Collaborator

No description provided.

@fredliang44 fredliang44 merged commit bd1e311 into ChatGPTNextWeb:main Dec 24, 2023
@@ -9,7 +9,9 @@ const serverConfig = getServerSideConfig();
export async function requestOpenai(req: NextRequest) {
const controller = new AbortController();

const authValue = req.headers.get("Authorization") ?? "";
const authValue =

Choose a reason for hiding this comment

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

fyi, this breaks the app for openai API - thanks!

@Cow258
Copy link

Cow258 commented Dec 25, 2023

the auth broken after this update,
POST - /api/openai/v1/chat/completions
401 Unauthorized

image
螢幕截圖 2023-12-25 08 41 24
image

@H0llyW00dzZ
Copy link
Contributor

H0llyW00dzZ commented Dec 25, 2023

yeah it's so bad, it just merge a bug to main lmao

@wenwenwen888
Copy link

改动后报错了

@Issues-translate-bot
Copy link

Bot detected the issue body's language is not English, translate it automatically.


An error was reported after the change

@H0llyW00dzZ
Copy link
Contributor

改动后报错了

#3628 (comment)

@Issues-translate-bot
Copy link

Bot detected the issue body's language is not English, translate it automatically.


An error was reported after the change

#3628 (comment)

spongeBor pushed a commit to spongeBor/ChatGPT-Next-Web that referenced this pull request Jan 18, 2024
fix: removing `bearer` from auth header when using azure
gaogao1030 pushed a commit to gaogao1030/ChatGPT-Next-Web that referenced this pull request May 16, 2024
fix: removing `bearer` from auth header when using azure
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants