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 chatgpt #772

Merged
merged 2 commits into from
Apr 14, 2024
Merged

Fix chatgpt #772

merged 2 commits into from
Apr 14, 2024

Conversation

asyncButNeverAwaits
Copy link
Contributor

@asyncButNeverAwaits asyncButNeverAwaits commented Mar 29, 2024

Close: #764

Refer to #764 (comment)

3.5 tested

4.0 NOT tested as no subscription

Copy link

vercel bot commented Mar 29, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
chatall ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 14, 2024 0:59am

@eddieai
Copy link

eddieai commented Mar 30, 2024

Thanks!
Can we merge it and release soon?

@sunner
Copy link
Owner

sunner commented Mar 30, 2024

Thank you for the contribution, but it does not work (both 3.5 and 4.0) on my side.

ChatGPT changed to wss:// protocol to transfer response several weeks ago.

@asyncButNeverAwaits
Copy link
Contributor Author

Thank you for the contribution, but it does not work (both 3.5 and 4.0) on my side.

ChatGPT changed to wss:// protocol to transfer response several weeks ago.

from my side, chatgpt is still using event stream

@eddieai
Copy link

eddieai commented Apr 3, 2024

Just checked on myside, ChatGPT 3.5 website is also using event-stream:
image

@sunner
Copy link
Owner

sunner commented Apr 5, 2024

My ChatGPT rollbacked to event-stream. Maybe the wss:// is just a test.

But still the PR doesn't work. The following pic is the response data through event-stream.
image

Have you ever met this?

@asyncButNeverAwaits
Copy link
Contributor Author

My ChatGPT rollbacked to event-stream. Maybe the wss:// is just a test.

But still the PR doesn't work. The following pic is the response data through event-stream. image

Have you ever met this?

no, these changes still working on my side

@sunner sunner merged commit cdfc4d0 into sunner:main Apr 14, 2024
2 checks passed
@sunner
Copy link
Owner

sunner commented Apr 14, 2024

Let's merge it and see what's happened for others. Thank you.

@darkomenx
Copy link

I confirm that in my Windows 11 x64 OS ChatGPT run correctly.
Very Thanks you for your works and your software 👍

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.

[BUG] ChatGPT 不可用
4 participants