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

调整DeepLX的设置。 #188

Closed
wants to merge 4 commits into from
Closed

调整DeepLX的设置。 #188

wants to merge 4 commits into from

Conversation

hellofinch
Copy link
Contributor

添加了针对V1的设置,有些服务有access token可选。
V2就不管了,V2直接用DeepL就行。

@Byaidu
Copy link
Owner

Byaidu commented Dec 9, 2024

image

你这个header怎么和官方文档的还不太一样,真的测试过能用吗……

@hellofinch
Copy link
Contributor Author

image
看上去没啥问题?它token直接都送到url最后了,直接放在authorization里应该也没问题吧?

@Byaidu
Copy link
Owner

Byaidu commented Dec 9, 2024

我的意思是它token前面应该有”Bearer“的

@Byaidu
Copy link
Owner

Byaidu commented Dec 9, 2024

有没有公共服务可以测试一下

@hellofinch
Copy link
Contributor Author

加access token的是对应pro,没看到有公共服务?

@hellofinch
Copy link
Contributor Author

token换个位置,不在header里设置了,直接放在url里。

@Byaidu
Copy link
Owner

Byaidu commented Dec 10, 2024

那我建议还是别折腾了,沉浸式都不支持deeplx v2

@Byaidu Byaidu closed this Dec 10, 2024
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.

2 participants