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

实现 #2971 在游戏崩溃窗口添加上传崩溃信息功能 #3145

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

Conversation

YELANDAOKONG
Copy link

效果

image

image

@YELANDAOKONG YELANDAOKONG changed the title 实现 #2971: 在游戏崩溃窗口添加上传崩溃信息功能 实现 #2971 在游戏崩溃窗口添加上传崩溃信息功能 Jun 22, 2024
@burningtnt

This comment was marked as outdated.

@burningtnt burningtnt closed this Jun 23, 2024
@burningtnt burningtnt reopened this Jun 23, 2024
@burningtnt burningtnt requested a review from Glavo June 23, 2024 01:36
@zkitefly
Copy link
Member

崩溃页面的 上传游戏崩溃信息 点了一下就不能点了是正常的吗

Copy link
Member

@burningtnt burningtnt left a comment

Choose a reason for hiding this comment

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

我反对这一功能。

按照 HMCL 崩溃分析群的实践结果,直接上传文件比给 Paste Bin 查看起来效率更高。并且,游戏崩溃报告包括 version.json,minecraft.log,latest.log,hmcl.log 和 launch script。这些信息都是崩溃分析必须的内容。只让当事人上传一部分内容,会导致信息缺失,降低效率。

@zkitefly
Copy link
Member

我建议移除上传单个日志

可以弄上传启动器日志

还有 file.io 下载了就立刻删除文件,或许该换个平台

@allMagicNB
Copy link

allMagicNB commented Jun 23, 2024

file.io 支持感觉要出问题,下载了马上就会删掉,万一有个手欠的下载了导致大夫下载不了……

@YELANDAOKONG
Copy link
Author

YELANDAOKONG commented Jun 23, 2024

如果通过使用分割线的方式把一些必要的信息(如minecraft.log、latest.log、hmcl.log、启动脚本等)整合到同一个文件中上传到mclo.gs呢?类似这个效果:

image

image

这样就会得到一个包含许多必要信息的日志链接了

@zkitefly
Copy link
Member

如果通过使用分割线的方式把一些必要的信息(如minecraft.log、latest.log、hmcl.log、启动脚本等)整合到同一个文件中上传到mclo.gs呢?类似这个效果:

image

image

这样就会得到一个包含许多必要信息的日志链接了

mclo.gs 有行数限制吗?

@YELANDAOKONG
Copy link
Author

如果通过使用分割线的方式把一些必要的信息(如minecraft.log、latest.log、hmcl.log、启动脚本等)整合到同一个文件中上传到mclo.gs呢?类似这个效果:

image

image

这样就会得到一个包含许多必要信息的日志链接了

mclo.gs 有行数限制吗?

10MB 2.5万行

@zkitefly
Copy link
Member

如果通过使用分割线的方式把一些必要的信息(如minecraft.log、latest.log、hmcl.log、启动脚本等)整合到同一个文件中上传到mclo.gs呢?类似这个效果:
image
image
这样就会得到一个包含许多必要信息的日志链接了

mclo.gs 有行数限制吗?

10MB 2.5万行

有些人的日志(minecraft.log)会达到 300 mb 甚至更多

@YELANDAOKONG
Copy link
Author

mclo.gs 有行数限制吗?

10MB 2.5万行

有些人的日志(minecraft.log)会达到 300 mb 甚至更多

那就很抽象了...

@burningtnt
Copy link
Member

按照 HMCL 崩溃分析群的实践结果,直接上传文件比给 Paste Bin 查看起来效率更高。并且,游戏崩溃报告包括 version.json,minecraft.log,latest.log,hmcl.log 和 launch script。这些信息都是崩溃分析必须的内容。只让当事人上传一部分内容,会导致信息缺失,降低效率。

反对的核心在于“查看文件”比“看剪贴板”更方便

@Tinhone
Copy link

Tinhone commented Jun 25, 2024

是否有其他更好且可用的文件分享网站?🤔

@zkitefly
Copy link
Member

可以试试 https://litterbox.catbox.moe/

@3gf8jv4dv
Copy link
Contributor

可以试试 litterbox.catbox.moe

This one looks good.

@3gf8jv4dv
Copy link
Contributor

It would be a good idea if could just upload the exported log archive file to a temporary storage site. I personally think using separators to separate different types of logs in one file is a bit bad.

@zkitefly
Copy link
Member

注意一下,使用 https://litterbox.catbox.moe/ 时复制链接的内容建议加上该文件的名称,例如 xxxx-xxxx-xxxx-xxxx.zip: https://xxxx.xxxx,因为该平台下载文件的时候不会保留该文件的原始名称,但崩溃信息需要文件上的导出时间

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.

6 participants