research the issues with tests

Signed-off-by: Roman Dmytrenko <rdmytrenko@gmail.com>

build/testing/integration/authz/auth.go

@@ -281,6 +281,9 @@ func (s clientCallSet) assert(t *testing.T, ctx context.Context, client sdk.SDK)
 type clientCall func(*testing.T, context.Context, sdk.SDK) error
 
 func GetNamespace(in *flipt.GetNamespaceRequest) clientCall {
+	if in.GetNamespaceKey() == "" {
+		in.Key = "default"
+	}
 	return func(t *testing.T, ctx context.Context, s sdk.SDK) error {
 		_, err := s.Flipt().GetNamespace(ctx, in)
 		return fmt.Errorf("GetNamespace: %w", err)
@@ -434,6 +437,7 @@ func CreateRollout(in *flipt.CreateRolloutRequest) clientCall {
 		return fmt.Errorf("CreateRollout: %w", err)
 	}
 }
+
 func UpdateRollout(in *flipt.UpdateRolloutRequest) clientCall {
 	return func(t *testing.T, ctx context.Context, s sdk.SDK) error {
 		_, err := s.Flipt().UpdateRollout(ctx, in)

internal/server/authz/middleware/grpc/middleware.go

@@ -22,13 +22,11 @@ type InterceptorOptions struct {
 	skippedServers []any
 }
 
-var (
-	// methods which should always skip authorization
-	skippedMethods = map[string]any{
-		"/flipt.auth.AuthenticationService/GetAuthenticationSelf":    struct{}{},
-		"/flipt.auth.AuthenticationService/ExpireAuthenticationSelf": struct{}{},
-	}
-)
+// methods which should always skip authorization
+var skippedMethods = map[string]any{
+	"/flipt.auth.AuthenticationService/GetAuthenticationSelf":    struct{}{},
+	"/flipt.auth.AuthenticationService/ExpireAuthenticationSelf": struct{}{},
+}
 
 func skipped(ctx context.Context, info *grpc.UnaryServerInfo, o InterceptorOptions) bool {
 	// if we skip authentication then we must skip authorization
@@ -95,7 +93,6 @@ func AuthorizationRequiredInterceptor(logger *zap.Logger, policyVerifier authz.V
 				"request":        request,
 				"authentication": auth,
 			})
-
 			if err != nil {
 				logger.Error("unauthorized", zap.Error(err))
 				return ctx, errUnauthorized