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

安全问题! #46

Closed
undefinedplayer opened this issue Jun 19, 2019 · 3 comments
Closed

安全问题! #46

undefinedplayer opened this issue Jun 19, 2019 · 3 comments

Comments

@undefinedplayer
Copy link

里面太多的http了,很多都可以用https的。我在互联网安全公司工作,安全分析师刚刚给我看我查的每一个词在公司网络上都看得到。现在全公司都知道我英语不好了!泪...
而且代码里有把在线查到的词上传到你的服务器数据库是吗?这至少应该提前说一声啊。这也是隐私保护的一部分。最好应该可以选择愿不愿意上传,而不是只要用了这个工具就上传给你。

@ChestnutHeng
Copy link
Owner

收集词库有两个原因:

  1. 为了扩展离线词典的查询命中率,目前有1/4的查词无法在离线词典找到
  2. 缩减词典体积
    上传的词条只收集了词条本身,不关注用户,当然大家可以把这个上报文件注释掉~代码都在那里

@undefinedplayer
Copy link
Author

收集词库有两个原因:

  1. 为了扩展离线词典的查询命中率,目前有1/4的查词无法在离线词典找到
  2. 缩减词典体积
    上传的词条只收集了词条本身,不关注用户,当然大家可以把这个上报文件注释掉~代码都在那里

好的,如果能把里面的http能改成https的都改过去那就更好了。非常感谢这个项目。

@ChestnutHeng
Copy link
Owner

已经升级到https。请git pull origin master体验

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

No branches or pull requests

2 participants