From e023b6e51e7ebb2a553e1c1f3a1abacc22d72591 Mon Sep 17 00:00:00 2001 From: Alex Vanin Date: Wed, 21 Oct 2020 18:10:49 +0300 Subject: [PATCH] [#179] sdk/client: Ease bearer token sanity check Now owner ID field is not required to be set. According to latest neofs-api, this field set if token was issued for specific owner ID. If this field is not set, then any user can use this token while it is correctly signed and has valid lifetime. Lifetime is also can be omitted since node interpret empty lifetime as a lifetime with zero values. Signed-off-by: Alex Vanin --- pkg/token/bearer.go | 4 ---- 1 file changed, 4 deletions(-) diff --git a/pkg/token/bearer.go b/pkg/token/bearer.go index 13e26cb..579d055 100644 --- a/pkg/token/bearer.go +++ b/pkg/token/bearer.go @@ -95,12 +95,8 @@ func sanityCheck(b *BearerToken) error { return errors.New("bearer token is not set") case b.token.GetBody() == nil: return errors.New("bearer token body is not set") - case b.token.GetBody().GetLifetime() == nil: - return errors.New("bearer token lifetime is not set") case b.token.GetBody().GetEACL() == nil: return errors.New("bearer token EACL table is not set") - case b.token.GetBody().GetOwnerID() == nil: - return errors.New("bearer token owner is not set") } // consider checking EACL sanity there, lifetime correctness, etc.