Skip to content

Allow change Weather API URLs from HTTPS to HTTP #1569#1570

Merged
HelloWRC merged 5 commits into
ClassIsland:masterfrom
liuziheng20091106:master
Jan 29, 2026
Merged

Allow change Weather API URLs from HTTPS to HTTP #1569#1570
HelloWRC merged 5 commits into
ClassIsland:masterfrom
liuziheng20091106:master

Conversation

@liuziheng20091106

Copy link
Copy Markdown
Contributor

这个 Pull Request 做了什么?

修改获取天气的协议为http,让极个别网络环境下的电脑可以及时刷新天气信息

相关 Issue

Fixes #1569

检查清单

  • 我已经在本地测试过这个 PR,确保欲实现的功能或修复的问题能正常工作。

@liuziheng20091106 liuziheng20091106 requested review from a team and HelloWRC as code owners December 29, 2025 14:30
@MKStoler1024

Copy link
Copy Markdown

不该作为硬性要求,该作为可选设置项

@HelloWRC HelloWRC left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

出于安全考虑,无论传输的是什么数据,禁用 TLS 功能都不应该默认启用。请为此功能添加启用开关并设为默认禁用,并在设置开关中添加危险警示语。

@liuziheng20091106

This comment has been minimized.

@liuziheng20091106

Copy link
Copy Markdown
Contributor Author

想法更新:

要不加个自定义BaseURL的功能?
自己在学校建一个服务器,功能实现起来更自由()

本条评论点赞超过20就开写

Comment thread ClassIsland/Services/WeatherService.cs Outdated
Comment thread ClassIsland/Views/SettingPages/WeatherSettingsPage.axaml Outdated
@liuziheng20091106 liuziheng20091106 changed the title Change Weather API URLs from HTTPS to HTTP #1569 Allow change Weather API URLs from HTTPS to HTTP #1569 Jan 9, 2026
本版本由于无实质性修改,所以未经检验。
Comment thread ClassIsland/Views/SettingPages/WeatherSettingsPage.axaml
@liuziheng20091106

Copy link
Copy Markdown
Contributor Author

我现在严重怀疑我忘记提交一个文件了

涉及更改后请求重启
@liuziheng20091106

Copy link
Copy Markdown
Contributor Author

鉴于最近300年(夸张)上游分支中weather根本没人改,为了确保commit精简,本PR暂未同步上游更改。

合并冲突应该不可能,功能冲突更不可能,这就是挑选冷门功能pr的魅力()

@HelloWRC HelloWRC merged commit a71a6aa into ClassIsland:master Jan 29, 2026
2 checks passed
@HelloWRC

Copy link
Copy Markdown
Member

感谢您的贡献🤝

昔涟 爱

Koharu-Mizuki pushed a commit to Koharu-Mizuki/ClassIsland-RB that referenced this pull request Jun 19, 2026
… (ClassIsland#1570)

* Change API URLs from HTTPS to HTTP ClassIsland#1569

兼容性++

* 将TLS的禁用设置为选项

* code:优化可维护性
本版本由于无实质性修改,所以未经检验。

* 提交遗漏的文件
涉及更改后请求重启
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.

在无法使用TLS时,小米天气无法正常访问

3 participants