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

修复 因为媒体服务器不存在 而导致的 panic - 数组溢出 ,并且新增媒体服务器提示状态. #347

Closed
wants to merge 6 commits into from

Conversation

SudoSuBash
Copy link

修复 因为媒体服务器不存在 而导致的 panic - 数组溢出 ,并且新增媒体服务器提示状态
对应Issue:
Mrs4s/go-cqhttp#2517
Mrs4s/go-cqhttp#2521

现在可以和 Mrs4s/go-cqhttp#2522 一起配合检测媒体服务器是否存在.

client/internal/highway/highway.go Outdated Show resolved Hide resolved
* Written by Bash
* 没办法了,只有把媒体服务器地址写死在服务器里面,算是一种曲线救国
*/
Addre := [4]int{1936450177, 3211518593, 761732366, 993564539}
Copy link
Collaborator

Choose a reason for hiding this comment

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

所有变量名首字母小写。

Copy link
Author

@SudoSuBash SudoSuBash Feb 26, 2024

Choose a reason for hiding this comment

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

抱歉,请把我这个pull requests删除了吧,目前测试,这种方法在发送语音时接口仍会报错199,但是在转发消息的时候它正常工作了。
目前不知道有好的解决方案。

Copy link
Collaborator

Choose a reason for hiding this comment

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

ok.

Copy link
Collaborator

Choose a reason for hiding this comment

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

pr是删不掉的,只能close。

@@ -224,6 +224,28 @@ func (s *Session) connect(addr Addr) (persistConn, error) {
func (s *Session) nextAddr() Addr {
s.addrMu.Lock()
defer s.addrMu.Unlock()

if len(s.SsoAddr) == 0 {
//fmt.Println("test")
Copy link
Collaborator

Choose a reason for hiding this comment

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

删除不用的注释。

@SudoSuBash SudoSuBash closed this Feb 26, 2024
@fumiama
Copy link
Collaborator

fumiama commented Feb 26, 2024

你不用关掉再开PR,直接继续更改就可以了。

@fumiama fumiama reopened this Feb 26, 2024
@fumiama fumiama closed this Feb 29, 2024
@SinKingCloud
Copy link

经过测试可以调用UploadImage方法来让服务端主动返回server list。但是上传群文件只能上传txt类型的。估计协议已经更新了。需要抓包才能彻底修复。
image

@kkgg0521
Copy link

经过测试可以调用UploadImage方法来让服务端主动返回server list。但是上传群文件只能上传txt类型的。估计协议已经更新了。需要抓包才能彻底修复。 image

看了一下應該是丟包導致的

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.

4 participants