Skip to content

Commit 0d2f0d4

Browse files
authored
remove string slice and strings.join (#115)
1 parent 148d710 commit 0d2f0d4

File tree

1 file changed

+12
-13
lines changed

1 file changed

+12
-13
lines changed

token.go

+12-13
Original file line numberDiff line numberDiff line change
@@ -63,35 +63,34 @@ func NewWithClaims(method SigningMethod, claims Claims) *Token {
6363
// SignedString creates and returns a complete, signed JWT. The token is signed
6464
// using the SigningMethod specified in the token.
6565
func (t *Token) SignedString(key interface{}) (string, error) {
66-
var sig, sstr string
67-
var err error
68-
if sstr, err = t.SigningString(); err != nil {
66+
sstr, err := t.SigningString()
67+
if err != nil {
6968
return "", err
7069
}
71-
if sig, err = t.Method.Sign(sstr, key); err != nil {
70+
71+
sig, err := t.Method.Sign(sstr, key)
72+
if err != nil {
7273
return "", err
7374
}
74-
return strings.Join([]string{sstr, sig}, "."), nil
75+
76+
return sstr + "." + sig, nil
7577
}
7678

7779
// SigningString generates the signing string. This is the most expensive part
7880
// of the whole deal. Unless you need this for something special, just go
7981
// straight for the SignedString.
8082
func (t *Token) SigningString() (string, error) {
81-
var err error
82-
var jsonValue []byte
83-
84-
if jsonValue, err = json.Marshal(t.Header); err != nil {
83+
h, err := json.Marshal(t.Header)
84+
if err != nil {
8585
return "", err
8686
}
87-
header := EncodeSegment(jsonValue)
8887

89-
if jsonValue, err = json.Marshal(t.Claims); err != nil {
88+
c, err := json.Marshal(t.Claims)
89+
if err != nil {
9090
return "", err
9191
}
92-
claim := EncodeSegment(jsonValue)
9392

94-
return strings.Join([]string{header, claim}, "."), nil
93+
return EncodeSegment(h) + "." + EncodeSegment(c), nil
9594
}
9695

9796
// Parse parses, validates, verifies the signature and returns the parsed token.

0 commit comments

Comments
 (0)