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.

Sign up for GitHub

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

Jump to bottom

当用户尝试配置使用非 HTTPS 协议的验证服务器时，给予警告以让用户知晓可能存在的安全风险 #5439

Open
wants to merge 14 commits into
base: main
Choose a base branch
from
Open

当用户尝试配置使用非 HTTPS 协议的验证服务器时，给予警告以让用户知晓可能存在的安全风险 #5439

wants to merge 14 commits into from
+43 −2

Conversation

shimoranla
Copy link
Collaborator

#5357 重开

@3gf8jv4dv 3gf8jv4dv added 处理中 开发者正在调查或处理该项 · 新功能 🟩 低 优先度：低 labels Jan 18, 2025
@MoYuan-CN
Copy link
Collaborator

Tip

不要在一个 Pull Request 中贡献多个不同的内容

@shimoranla
Copy link
Collaborator Author

不要在一个 Pull Request 中贡献多个不同的内容

只是用于测试的提交而已，实际做的时候肯定会单独开的（

@shimoranla shimoranla requested a review from LTCatt January 20, 2025 02:37
@zkitefly zkitefly mentioned this pull request Jan 24, 2025
Closed
LTCatt
LTCatt requested changes Feb 12, 2025
View reviewed changes
Copy link
Member

@LTCatt LTCatt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

想了想，其实应该放在启动前检查的时候弹窗？
一方面情况较为罕见，没必要加 UI。
一方面按现在的这个处理方式，直接导入整合包的玩家是看不到这个提示的，放到启动前检查的时候玩家也能看到……

@LTCatt LTCatt added 等待提交者 需要提交者补充信息或进行操作 and removed 处理中 开发者正在调查或处理该项 labels Feb 12, 2025
@shimoranla
Copy link
Collaborator Author

shimoranla commented Feb 13, 2025 •
edited
Loading

🤔 等周末再说，今天刚好开学

Edit:最近忙于信息竞赛项目，可能还得推迟一点

Edit2:这周应该能写掉，周五完成作品之后应该有大把的时间

@shimoranla
Copy link
Collaborator Author

shimoranla commented Feb 27, 2025 •
edited
Loading

忙里偷闲改了一下

目前的实现是:

如果使用 http 协议，那么展示警告，用户如果按下不再显示，之后的登录不会再显示这个警告
但是如果用户修改了验证服务器地址，那么这个警告会被重新启用（考虑到部分验证服务器可能继续提供 HTTP 地址但是用户不知情）
考虑到账号安全的重要性，使用了和删除 Minecraft 文件夹一样的弹窗模式，两个取消启动+一个不再显示，高亮取消启动，避免用户无意识按下不再显示
如果没有勾选验证 SSL 证书，会显示提示建议用户启用这个选项
同样的，按下不再显示后不会继续显示，但修改验证服务器地址会被重新启用
这个选项是正常样式，如果觉得不行也可以再改改，不过我觉得这可能被用户不看文本就当未找到 Java 按下去

这两个检查会在正式登录前进行

龙猫再看看？

@shimoranla shimoranla requested a review from LTCatt February 27, 2025 04:58
@3gf8jv4dv 3gf8jv4dv added 等待确认 已经过社区确认，等待开发者确认 and removed 等待提交者 需要提交者补充信息或进行操作 labels Feb 27, 2025
@LTCatt LTCatt added 处理中 开发者正在调查或处理该项 and removed 等待确认 已经过社区确认，等待开发者确认 labels Mar 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@LTCatt LTCatt Awaiting requested review from LTCatt

Assignees
No one assigned
Labels
· 新功能 处理中 开发者正在调查或处理该项 🟩 低 优先度：低
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@shimoranla @MoYuan-CN @LTCatt @3gf8jv4dv @wuliaodexiaoluo