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: remove the content-encoding header #3215

Merged
merged 2 commits into from
Dec 26, 2023
Merged

Conversation

akira-cn
Copy link
Contributor

@akira-cn akira-cn commented Nov 10, 2023

The latest version of the OpenAI API forced the content-encoding to be brotli in json response so if the streaming is disabled, we need to remove the content-encoding header from the response of openai.
This is because Vercel uses gzip to compress the response data, if we don't remove the content-encoding header the browser will try to decode the response with brotli and cause a decompression fail.

image

@H0llyW00dzZ
Copy link
Contributor

++++

@Yidadaa Yidadaa merged commit 350ddd2 into ChatGPTNextWeb:main Dec 26, 2023
H0llyW00dzZ pushed a commit to H0llyW00dzZ/ChatGPT-Next-Web that referenced this pull request Dec 26, 2023
fix: remove the content-encoding header
H0llyW00dzZ added a commit to H0llyW00dzZ/ChatGPT-Next-Web that referenced this pull request Dec 26, 2023
* Merge pull request ChatGPTNextWeb#3215 from akira-cn/main

fix: remove the content-encoding header

* Update Web App link in README.md

* Update CN web app link in README.md

* Merge pull request ChatGPTNextWeb#3570 from Rocky0102/main

* chore: update auth value logic (ChatGPTNextWeb#3630)

* Fix [Api] [Common Server Side] Change variable

- [+] fix(common.ts): change variable declaration from 'var' to 'let' for 'authValue'

* Fix Front End Issue about package

- [+] chore(package.json): update "@tauri-apps/api" dependency to version 1.5.3
- [+] chore(yarn.lock): update "@tauri-apps/api" dependency to version 1.5.3

---------

Co-authored-by: Yidadaa <yidadaa@qq.com>
Co-authored-by: ChatGPTNextWeb <153288546+ChatGPTNextWeb@users.noreply.github.com>
Co-authored-by: fredliang44 <info@fredliang.cn>
spongeBor pushed a commit to spongeBor/ChatGPT-Next-Web that referenced this pull request Jan 18, 2024
fix: remove the content-encoding header
gaogao1030 pushed a commit to gaogao1030/ChatGPT-Next-Web that referenced this pull request May 16, 2024
fix: remove the content-encoding header
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.

3 participants