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

fix: panic occurred when receive voice record in group message #349

Closed
wants to merge 10 commits into from

Conversation

Fripine
Copy link

@Fripine Fripine commented Sep 3, 2024

Protocol: Android Pad 9.0.90.18945

需要Uid支持才能修复语音的接收和下载 #350

已经完成对NTQQ图片链接的修复,以及接收语音消息字段为空值的问题(需要在gocqhttp上进一步修改)。
因为ParseMessageElems没办法传client进来,而语音消息的url字段需要通过发OidbSvcTrpcTcp.0x126d_200包来获取,所以在ParseMessageElems不进行url字段的赋值,保留FileId用作后续发包使用

proto文件新字段命名参考了lagrangeGo

message/message.go Outdated Show resolved Hide resolved
@Fripine Fripine marked this pull request as draft September 4, 2024 10:29
@Fripine
Copy link
Author

Fripine commented Sep 4, 2024

吐槽一下这个PbMultiMediaElement感觉就不是自动生成的,像是手打上去的,我用protoc-gen-golite生成的格式完全不一样,以为搞坏了

@fumiama
Copy link
Collaborator

fumiama commented Sep 4, 2024

吐槽一下这个PbMultiMediaElement感觉就不是自动生成的,像是手打上去的,我用protoc-gen-golite生成的格式完全不一样,以为搞坏了

用的是修改过的github.com/RomiChan/protobuf/cmd/protoc-gen-golite。详见根目录Makefile

@Fripine
Copy link
Author

Fripine commented Sep 5, 2024

吐槽一下这个PbMultiMediaElement感觉就不是自动生成的,像是手打上去的,我用protoc-gen-golite生成的格式完全不一样,以为搞坏了

用的是修改过的github.com/RomiChan/protobuf/cmd/protoc-gen-golite。详见根目录Makefile

我就是按照makefile里面的做的,安装了里面的特定版本,并且按照里面写的生成指令生成的,和原有的pb.go不一样
如果提交不合规的话,可能得麻烦开发组帮忙弄一下这个,第一次用,搞不清楚为什么

@wdvxdr1123
Copy link
Collaborator

应该是 #343 这里没有使用工具生成代码

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.

3 participants