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

Support downloading OptiFine from it's own maven. #2719

Open
wants to merge 7 commits into
base: javafx
Choose a base branch
from
Open

Support downloading OptiFine from it's own maven. #2719

wants to merge 7 commits into from

Conversation

burningtnt
Copy link
Member

@burningtnt burningtnt commented Feb 1, 2024 •
edited

Close #2717

目前更新 JSON 为 https://zkitefly.github.io/optifine-download-list/index.json
其中包含 OF-302 服务器的位置和各版本信息

如果接受这个方案，可以考虑将 optifine-download-list 这个仓库 fork 一份到 HMCL-dev 组织下，然后转移链接

问题：OptiFine 的仓库不包含所有版本，会比原先的 BMCLAPI 少文件

@burningtnt burningtnt changed the title Support download OptiFine from it's own maven. Support downloading OptiFine from it's own maven. Feb 1, 2024
@zkitefly
Copy link
Member

zkitefly commented Feb 1, 2024

为防止整合包安装等搜索版本号搜不到的情况，可以增加一个规则，如果在整合包安装等搜索版本号搜不到自动切换至BMCLAPI尝试搜索

@zkitefly zkitefly requested a review from Glavo February 1, 2024 14:49
@zkitefly
Copy link
Member

zkitefly commented Feb 1, 2024 •
edited

还有一种方案，将 BMCLAPI 中获取额外文件放入 index.json，利用 OF-302，如果未找到该文件，则自动通过正则表达式匹配文本返回 BMCLAPI 的下载链接

@zkitefly
Copy link
Member

zkitefly commented Feb 1, 2024

已向 optifine-downloadfile-302 添加功能 若 OptiFine 官网无此文件，则跳转至 BMCLAPI 下载文件

数据少的问题解决了

@zkitefly
Copy link
Member

zkitefly commented Feb 3, 2024 •
edited

已部署 Netlify 平台的托管，该平台更加稳定快速（国内国外都是）

https://optifine-download-list.netlify.app/index.json

image

image

@zkitefly
Copy link
Member

zkitefly commented Feb 3, 2024

burningtnt#4

@zkitefly
Copy link
Member

zkitefly commented Feb 3, 2024

_deepin-terminal_20240203005129.mp4

测试了一下，下载失败并不会自动跳转至下一个下载源

@zkitefly
Copy link
Member

zkitefly commented Feb 4, 2024

_deepin-terminal_20240203005129.mp4

测试了一下，下载失败并不会自动跳转至下一个下载源

解决一下？

@burningtnt

zkitefly and others added 2 commits February 5, 2024 01:03
@burningtnt
Copy link
Member Author

burningtnt commented Feb 8, 2024 •
edited

MCBBS 没关站的时候提供 OptiFine 的下载吗？还是说，一直只有 BMCLAPI 提供？

  • 如果提供了，那么等到 MCBBS 好了，本 PR 可以考虑改为和 Prism Launcher 一样，让用户自己去下载，然后拖入启动器继续逻辑，从而避免法律风险
  • 如果没有提供，那么需要论证一下必要性。非中国大陆地区连接 BMCLAPI 的速度怎么样？

@hejiehao
Copy link
Contributor

hejiehao commented Feb 8, 2024

MCBBS 没关站的时候提供 OptiFine 的下载吗？还是说，一直只有 BMCLAPI 提供？

  • 如果提供了，那么等到 MCBBS 好了，本 PR 可以考虑改为和 Prism Launcher 一样，让用户自己去下载，然后拖入启动器继续逻辑，从而避免法律风险
  • 如果没有提供，那么需要论证一下必要性。非中国大陆地区连接 BMCLAPI 的速度怎么样？

不是说mcbbs源是BMCLAPI的一个特别镜像吗？

@burningtnt
Copy link
Member Author

不是说 MCBBS 源是BMCLAPI的一个特别镜像吗？

是的，但我怕 MCBBS 和高校一样只处理部分资源

@zkitefly
Copy link
Member

zkitefly commented Feb 8, 2024

不是说 MCBBS 源是BMCLAPI的一个特别镜像吗？

是的，但我怕 MCBBS 和高校一样只处理部分资源

image

@zkitefly
Copy link
Member

zkitefly commented Feb 8, 2024

MCBBS 没关站的时候提供 OptiFine 的下载吗？还是说，一直只有 BMCLAPI 提供？

  • 如果提供了，那么等到 MCBBS 好了，本 PR 可以考虑改为和 Prism Launcher 一样，让用户自己去下载，然后拖入启动器继续逻辑，从而避免法律风险
  • 如果没有提供，那么需要论证一下必要性。非中国大陆地区连接 BMCLAPI 的速度怎么样？

这样的话，还不如保持现在的下载逻辑

@burningtnt burningtnt marked this pull request as draft February 17, 2024 04:14
@burningtnt
Copy link
Member Author

需要观察一段时间

@zkitefly
Copy link
Member

zkitefly commented Feb 24, 2024 •
edited

我认为这个 PR 有一定的重要性，因为有时候 BMCLAPI 会因为 DDOS 等攻击暂时下线，会影响依赖 BMCLAPI 的功能

而且目前 PCL 已经爬 OptiFine 官网下载很久了，也没见出什么事，我认为不会出什么岔子（

@burningtnt burningtnt marked this pull request as ready for review February 24, 2024 07:50
@burningtnt
Copy link
Member Author

根据近期观察，以及 Zkitefly 的提醒，我觉得这个功能是有必要的。
关于 index.json，我觉得可以保持现在托管在 GitHub Pages 上。反正也要从 OptiFine 官网拉数据，那就没必要部署到国内服务器

@burningtnt
Copy link
Member Author

本 PR 中的功能已被合并入 PR Collection 供测试

@burningtnt
Copy link
Member Author

近期 BMCLAPI 服务趋于稳定，即使国外走这里，效果也是不错的。继续观察

@burningtnt burningtnt marked this pull request as draft March 11, 2024 11:30
@burningtnt
Copy link
Member Author

被新加坡一带的用户问候了，看起来这个 PR 还是有意义的

@burningtnt burningtnt marked this pull request as ready for review May 5, 2024 10:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@huanghongxun huanghongxun Awaiting requested review from huanghongxun

@Glavo Glavo Awaiting requested review from Glavo

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Feature] 添加一个新的 optifine 下载源
3 participants
@burningtnt @zkitefly @hejiehao