Skip to content

Commit 8803cc0

Browse files
authored
Merge pull request #179 from q956673351/fix/nil_pointer_err
fix: #178 goroutine nil pointer causes the service crash
2 parents 84a9657 + a5e036f commit 8803cc0

File tree

1 file changed

+6
-5
lines changed

1 file changed

+6
-5
lines changed

code/handlers/msg.go

+6-5
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ import (
55
"context"
66
"encoding/base64"
77
"fmt"
8+
"errors"
89
"github.com/google/uuid"
910
larkcard "github.com/larksuite/oapi-sdk-go/v3/card"
1011
larkim "github.com/larksuite/oapi-sdk-go/v3/service/im/v1"
@@ -67,7 +68,7 @@ func replyCard(ctx context.Context,
6768
// 服务端错误处理
6869
if !resp.Success() {
6970
fmt.Println(resp.Code, resp.Msg, resp.RequestId())
70-
return err
71+
return errors.New(resp.Msg)
7172
}
7273
return nil
7374
}
@@ -432,7 +433,7 @@ func replyMsg(ctx context.Context, msg string, msgId *string) error {
432433
// 服务端错误处理
433434
if !resp.Success() {
434435
fmt.Println(resp.Code, resp.Msg, resp.RequestId())
435-
return err
436+
return errors.New(resp.Msg)
436437
}
437438
return nil
438439
}
@@ -461,7 +462,7 @@ func uploadImage(base64Str string) (*string, error) {
461462
// 服务端错误处理
462463
if !resp.Success() {
463464
fmt.Println(resp.Code, resp.Msg, resp.RequestId())
464-
return nil, err
465+
return nil, errors.New(resp.Msg)
465466
}
466467
return resp.Data.ImageKey, nil
467468
}
@@ -495,7 +496,7 @@ func replyImage(ctx context.Context, ImageKey *string,
495496
// 服务端错误处理
496497
if !resp.Success() {
497498
fmt.Println(resp.Code, resp.Msg, resp.RequestId())
498-
return err
499+
return errors.New(resp.Msg)
499500
}
500501
return nil
501502

@@ -580,7 +581,7 @@ func sendMsg(ctx context.Context, msg string, chatId *string) error {
580581
// 服务端错误处理
581582
if !resp.Success() {
582583
fmt.Println(resp.Code, resp.Msg, resp.RequestId())
583-
return err
584+
return errors.New(resp.Msg)
584585
}
585586
return nil
586587
}

0 commit comments

Comments
 (0)