From e23f00f349706ad646390a812da20733567f9031 Mon Sep 17 00:00:00 2001 From: Andy Hsu Date: Sat, 10 Jun 2023 00:12:04 +0800 Subject: [PATCH] fix(139): avoid panic due to `Authorization` for emptiness --- drivers/139/driver.go | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/drivers/139/driver.go b/drivers/139/driver.go index 659e13aa..c8935b2c 100644 --- a/drivers/139/driver.go +++ b/drivers/139/driver.go @@ -32,12 +32,19 @@ func (d *Yun139) GetAddition() driver.Additional { } func (d *Yun139) Init(ctx context.Context) error { + if d.Authorization == "" { + return fmt.Errorf("authorization is empty") + } decode, err := base64.StdEncoding.DecodeString(d.Authorization) if err != nil { return err } decodeStr := string(decode) - d.Account = strings.Split(decodeStr, ":")[1] + splits := strings.Split(decodeStr, ":") + if len(splits) < 2 { + return fmt.Errorf("authorization is invalid, splits < 2") + } + d.Account = splits[1] _, err = d.post("/orchestration/personalCloud/user/v1.0/qryUserExternInfo", base.Json{ "qryUserExternInfoReq": base.Json{ "commonAccountInfo": base.Json{