Merge pull request #1953 from opengovern/fix-web-ui

fix: update analytics and queries
opengovern · Nov 12, 2024 · 5e50859 · 5e50859
2 parents c404ad8 + 42e482b
commit 5e50859
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/services/integration/client/integration.go b/services/integration/client/integration.go
@@ -16,7 +16,7 @@ type IntegrationServiceClient interface {
 	ListIntegrationsByFilters(ctx *httpclient.Context, req models.ListIntegrationsRequest) (*models.ListIntegrationsResponse, error)
 	IntegrationHealthcheck(ctx *httpclient.Context, integrationID string) (*models.Integration, error)
 	GetCredential(ctx *httpclient.Context, credentialID string) (*models.Credential, error)
-	ListCredentials(ctx *httpclient.Context) (models.ListCredentialsResponse, error)
+	ListCredentials(ctx *httpclient.Context) (*models.ListCredentialsResponse, error)
 	GetIntegrationGroup(ctx *httpclient.Context, integrationGroupName string) (*models.IntegrationGroup, error)
 	ListIntegrationGroups(ctx *httpclient.Context) ([]models.IntegrationGroup, error)
 }
@@ -96,7 +96,7 @@ func (c *integrationClient) GetCredential(ctx *httpclient.Context, credentialID
 	return response, nil
 }
 
-func (c *integrationClient) ListCredentials(ctx *httpclient.Context) (models.ListCredentialsResponse, error) {
+func (c *integrationClient) ListCredentials(ctx *httpclient.Context) (*models.ListCredentialsResponse, error) {
 	url := fmt.Sprintf("%s/api/v1/credentials", c.baseURL)
 	var response models.ListCredentialsResponse
 
@@ -106,7 +106,7 @@ func (c *integrationClient) ListCredentials(ctx *httpclient.Context) (models.Lis
 		}
 		return nil, err
 	}
-	return response, nil
+	return &response, nil
 }
 
 func (c *integrationClient) IntegrationHealthcheck(ctx *httpclient.Context, integrationID string) (*models.Integration, error) {

diff --git a/services/migrator/job/migrations/analytics/populate.go b/services/migrator/job/migrations/analytics/populate.go
@@ -54,6 +54,7 @@ func (m Migration) Run(ctx context.Context, conf config.MigratorConfig, logger *
 		return nil
 	})
 	if err != nil && !errors.Is(err, fs.ErrNotExist) {
+		logger.Error("failed to get assets", zap.Error(err))
 		return err
 	}
 
@@ -64,6 +65,7 @@ func (m Migration) Run(ctx context.Context, conf config.MigratorConfig, logger *
 		return nil
 	})
 	if err != nil && !errors.Is(err, fs.ErrNotExist) {
+		logger.Error("failed to get spends", zap.Error(err))
 		return err
 	}
 
@@ -74,6 +76,7 @@ func (m Migration) Run(ctx context.Context, conf config.MigratorConfig, logger *
 		return nil
 	})
 	if err != nil && !errors.Is(err, fs.ErrNotExist) {
+		logger.Error("failed to get queries", zap.Error(err))
 		return err
 	}