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: Skip JSON deserialization when accessing transcriptions and translations. #718

Merged
merged 2 commits into from
Nov 19, 2023

Conversation

techotaku
Copy link
Contributor

@techotaku techotaku commented Nov 18, 2023

close #719
只是临时修复,将之前移除的if加了回来,让这两个接口能用。
彻底解决需要common.UnmarshalBodyReusable能同时支持application/jsonmultipart/form-data

已经在我的环境里测试通过。

@techotaku
Copy link
Contributor Author

techotaku commented Nov 18, 2023

close #719

@songquanpeng songquanpeng merged commit 969f539 into songquanpeng:main Nov 19, 2023
@songquanpeng
Copy link
Owner

Thx~

@techotaku techotaku deleted the fix/audio branch November 19, 2023 08:15
dirname pushed a commit to dirname/one-api that referenced this pull request Nov 24, 2023
feat: support cloudflare gateway for azure (songquanpeng#666)

* 🐛 Fix cloudflare gateway request failure

* 🐛 fix channel test url error

fix: skip JSON deserialization when accessing transcriptions and translations (songquanpeng#718)

* fix: Skip JSON deserialization when accessing transcriptions and translations.

* chore: update impl

---------

docs: update readme (songquanpeng#724)

feat: update request struct (close songquanpeng#708)

refactor: remove consumeQuota  related logic (songquanpeng#738)

* feat: 删除relay-text中的consumeQuota变量

该变量始终为true,可以删除

* chore: remove useless code

---------

fix: add Message-ID header for email  (songquanpeng#732)

* feat: Add Message-ID to email headers to comply with RFC 5322

- Extract domain from SMTPFrom
- Generate a unique Message-ID
- Add Message-ID to email headers

* chore: check slice length

---------

fix: add Date header for email (songquanpeng#742)

* 修复自建邮箱发送错误: INVALID HEADER Missing required header field: "Date"

* chore: fix style

---------

feat: support Azure's Whisper model (songquanpeng#720)

docs: add pr template

docs: update readme
Co-Authored-By: JustSong <songquanpeng@foxmail.com>
Co-Authored-By: liyujie <29959257@qq.com>
Co-Authored-By: JustSong <39998050+songquanpeng@users.noreply.github.com>
dirname pushed a commit to dirname/one-api that referenced this pull request Nov 24, 2023
…slations (songquanpeng#718)

* fix: Skip JSON deserialization when accessing transcriptions and translations.

* chore: update impl

---------

Co-authored-by: JustSong <songquanpeng@foxmail.com>
dirname pushed a commit to dirname/one-api that referenced this pull request Nov 24, 2023
…slations (songquanpeng#718)

* fix: Skip JSON deserialization when accessing transcriptions and translations.

* chore: update impl

---------

Co-authored-by: JustSong <songquanpeng@foxmail.com>
greeeds pushed a commit to greeeds/one-api that referenced this pull request Mar 3, 2024
…slations (songquanpeng#718)

* fix: Skip JSON deserialization when accessing transcriptions and translations.

* chore: update impl

---------

Co-authored-by: JustSong <songquanpeng@foxmail.com>
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

2 participants