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

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

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

Conversation

shimoranla
Copy link

@shimoranla shimoranla commented Jan 1, 2025

根据 Yggdrasil 启动器技术规范#对非 HTTPS 验证服务器的警告 当用户尝试配置非 HTTPS 协议的地址时,启动器应当给予警告

@3gf8jv4dv 3gf8jv4dv added · 新功能 等待确认 已经过社区确认,等待开发者确认 labels Jan 1, 2025
@3gf8jv4dv 3gf8jv4dv requested a review from LTCatt January 1, 2025 11:31
@shimoranla shimoranla changed the title 对使用 HTTP 协议登录的用户进行警告 当用户尝试配置使用非 HTTPS 协议的验证服务器时,给予警告以让用户知晓使用 HTTP 协议的安全风险 Jan 1, 2025
@shimoranla shimoranla changed the title 当用户尝试配置使用非 HTTPS 协议的验证服务器时,给予警告以让用户知晓使用 HTTP 协议的安全风险 当用户尝试配置使用非 HTTPS 协议的验证服务器时,给予警告以让用户知晓可能存在的安全风险 Jan 1, 2025
@LTCatt LTCatt added 处理中 开发者正在调查或处理该项 🟩 低 优先度:低 and removed 等待确认 已经过社区确认,等待开发者确认 labels Jan 2, 2025
@Silverteal
Copy link
Contributor

考虑到 禁用 SSL 验证 启动器选项的存在,可能需要更多判断逻辑

@shimoranla shimoranla requested a review from LTCatt January 5, 2025 12:14
@shimoranla
Copy link
Author

shimoranla commented Jan 5, 2025

现在有个 Bug,第一次点进页面不会有任何提示条,但是第二次点进去或者稍微改下文本就会出现。。
由于我不会写 VB.NET ,也看不出来逻辑上有啥问题,反正看的一头雾水的(

Edit:现在支持对 HTTP 协议和 HTTPS 协议但是关闭证书校验两种情况展示不同的提示条,但是因为启动器默认配置是不校验的,所以没有启用校验的提示条是蓝色的以避免可能给用户带来理解成本

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
· 新功能 处理中 开发者正在调查或处理该项 🟩 低 优先度:低
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants