[qfix] Add return statement in getSpiffeIDFromPath func (#1402)

* [qfix] add return statement in getSpiffeIDFromPath Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com> * use Warnf Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com> Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>
networkservicemesh · Dec 19, 2022 · 465077a · 465077a
1 parent 554e6ac
commit 465077a
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/pkg/registry/common/authorize/common.go b/pkg/registry/common/authorize/common.go
@@ -75,6 +75,7 @@ func getRawMap(m *PathIdsMap) map[string][]string {
 func getSpiffeIDFromPath(ctx context.Context, path *grpcmetadata.Path) spiffeid.ID {
 	if len(path.PathSegments) == 0 {
 		log.FromContext(ctx).Warn("can't get spiffe id from empty path")
+		return spiffeid.ID{}
 	}
 	tokenString := path.PathSegments[0].Token
 
@@ -98,7 +99,7 @@ func getSpiffeIDFromPath(ctx context.Context, path *grpcmetadata.Path) spiffeid.
 
 	id, err := spiffeid.FromString(subString)
 	if err != nil {
-		log.FromContext(ctx).Warn("failed to parse spiffeid from string: %s", err.Error())
+		log.FromContext(ctx).Warnf("failed to parse spiffeid from string: %s", err.Error())
 		return spiffeid.ID{}
 	}
 	return id