Skip to content

Commit dea8c4b

Browse files
committed
refactor(jsapi): simplify signature generation in GetConfig and GetAgentConfig methods
- Replaced the use of strconv to format the signature string with fmt.Sprintf for improved readability. - Updated the signature generation logic in both GetConfig and GetAgentConfig methods to use a single formatted string. This change enhances code clarity and maintains functionality in the signature generation process.
1 parent 1ee33a0 commit dea8c4b

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

work/jsapi/jsapi.go

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,7 @@
11
package jsapi
22

33
import (
4-
"strconv"
5-
4+
"fmt"
65
"github.com/silenceper/wechat/v2/credential"
76
"github.com/silenceper/wechat/v2/util"
87
"github.com/silenceper/wechat/v2/work/context"
@@ -49,7 +48,8 @@ func (js *Js) GetConfig(uri string) (config *Config, err error) {
4948
}
5049
config.NonceStr = util.RandomStr(16)
5150
config.Timestamp = util.GetCurrTS()
52-
config.Signature = util.Signature(ticketStr, config.NonceStr, strconv.FormatInt(config.Timestamp, 10), uri)
51+
str := fmt.Sprintf("jsapi_ticket=%s&noncestr=%s&timestamp=%d&url=%s", ticketStr, config.NonceStr, config.Timestamp, uri)
52+
config.Signature = util.Signature(str)
5353
return
5454
}
5555

@@ -68,6 +68,7 @@ func (js *Js) GetAgentConfig(uri string) (config *Config, err error) {
6868
}
6969
config.NonceStr = util.RandomStr(16)
7070
config.Timestamp = util.GetCurrTS()
71-
config.Signature = util.Signature(ticketStr, config.NonceStr, strconv.FormatInt(config.Timestamp, 10), uri)
71+
str := fmt.Sprintf("jsapi_ticket=%s&noncestr=%s&timestamp=%d&url=%s", ticketStr, config.NonceStr, config.Timestamp, uri)
72+
config.Signature = util.Signature(str)
7273
return
7374
}

0 commit comments

Comments
 (0)