aarifullin/frostfs-api-go
1
0
Fork 0
forked from TrueCloudLab/frostfs-api-go
Code Pull requests Activity

[#199] sdk/token: Correct linter's remarks

Browse source 
Signed-off-by: Leonard Lyubich <leonard@nspcc.ru>
This commit is contained in:
Leonard Lyubich 2020-11-16 17:58:07 +03:00 committed by Alex Vanin
parent c0de2bf9e5
commit 00ce980d82
1 changed files with 11 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
pkg/token/bearer.go
View file

@ -13,6 +13,12 @@ import (
crypto "github.com/nspcc-dev/neofs-crypto"
)
var (
errNilBearerToken = errors.New("bearer token is not set")
errNilBearerTokenBody = errors.New("bearer token body is not set")
errNilBearerTokenEACL = errors.New("bearer token EACL table is not set")
)
type BearerToken struct {
token acl.BearerToken
}
@ -63,6 +69,7 @@ func (b *BearerToken) SignToken(key *ecdsa.PrivateKey) error {
}
signWrapper := v2signature.StableMarshalerWrapper{SM: b.token.GetBody()}
return signature.SignDataWithHandler(key, signWrapper, func(key []byte, sig []byte) {
bearerSignature := new(refs.Signature)
bearerSignature.SetKey(key)
@ -103,15 +110,15 @@ func NewBearerTokenFromV2(v2 *acl.BearerToken) *BearerToken {
}
}
// sanityCheck if bearer token is ready to be issued
// sanityCheck if bearer token is ready to be issued.
func sanityCheck(b *BearerToken) error {
switch {
case b == nil:
return errors.New("bearer token is not set")
return errNilBearerToken
case b.token.GetBody() == nil:
return errors.New("bearer token body is not set")
return errNilBearerTokenBody
case b.token.GetBody().GetEACL() == nil:
return errors.New("bearer token EACL table is not set")
return errNilBearerTokenEACL
}
// consider checking EACL sanity there, lifetime correctness, etc.