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: deeplx ICounts calculation mistake #711

Merged
merged 1 commit into from
Feb 8, 2024
Merged

fix: deeplx ICounts calculation mistake #711

merged 1 commit into from
Feb 8, 2024

Conversation

Maasea
Copy link
Contributor

@Maasea Maasea commented Feb 8, 2024

在 Deepl 的编码过程中通过计算翻译内容中的 i 的数量来防止被简单的重放攻击。

然而之前提交的计算方式直接将翻译内容中的 i 为 0 的情况 return 了,这导致如果请求翻译的内容不包含 i 字母,则翻译过程被终止无法请求。

@Maasea
Copy link
Contributor Author

Maasea commented Feb 8, 2024

另外,我更新到 Zotero 7 的时候发现,翻译侧边栏消失了。

没有去了解 zotero 7 的新特性,所以想问一下这是 bug 嘛?

image

@windingwind
Copy link
Owner

感谢pr。

关于z7>beta55的侧边栏问题,须待zotero的新侧边栏api可用后进行修复。

@windingwind windingwind merged commit d357122 into windingwind:main Feb 8, 2024
Copy link

github-actions bot commented Jul 6, 2024

🚀 This ticket has been resolved in v1.1.0-beta.50. See Release v1.1.0-beta.50 for release notes.

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