Skip to content

Conversation

@StacieClark-Elastic
Copy link
Member

Fixed a bug where a panel in the Login Dashboard did not filter correctly. The 'Successful Logins by Compromised Users [Logs Google Workspace]' panel showed all logins. The panel was renamed to accurately describe the data being shown after a filter for google_workspace.login.is_suspicious: true was added. Updated ingest to store google_workspace.login data that are boolValue to capture is_suspicious flag on logins. Changed name of panel 'Successful Logins by Compromised Users [Logs Google Workspace]' to 'Successful Logins that are Suspicious [Logs Google Workspace]'. Added filter for google_workspace.login.is_suspicious: true to renamed panel.

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.
  • I have verified that any added dashboard complies with Kibana's Dashboard good practices

How to test this PR locally

Load the test data into the local elastic stack:

In the google_workspace package directory

> elastic-package build start the stack: elastic-package stack up -v --version=8.17.1 (wait for it to finish) then load the data > elastic-package test system -v --config-file data_stream/login/_dev/test/system/test-default-config.yml --setup > elastic-package test system -v --no-provision Open the dashboard: Kibana->Security->Dashboards->[Logs Google Workspace] Login Verify that the panel 'Successful Logins by Compromised Users [Logs Google Workspace]' exist Verify that a single entry for a successful login exists. Open the record and verify that the entry has the google_workspace.login.is_suspicious flag set to true Tear down the integration (failure to do this step will leave data on your local filesystem) > elastic-package test system -v --tear-down stop the stack: elastic-package stack down 

Related issues

@StacieClark-Elastic StacieClark-Elastic added bug Something isn't working, use only for issues Integration:google_workspace Google Workspace Team:Service-Integrations Label for the Observability Service Integrations team labels Feb 5, 2025
Fixed a bug where a panel in the Login Dashboard did not filter correctly. The 'Successful Logins by Compromised Users [Logs Google Workspace]' panel showed all logins. The panel was renamed to accurately describe the data being shown after a filter for google_workspace.login.is_suspicious: true was added. Updated ingest to store google_workspace.login data that are boolValue to capture is_suspicious flag on logins. Changed name of panel 'Successful Logins by Compromised Users [Logs Google Workspace]' to 'Successful Logins that are Suspicious [Logs Google Workspace]'. Added filter for google_workspace.login.is_suspicious: true to renamed panel.
@andrewkroh andrewkroh force-pushed the bugfix/add-missing-filter-to-a-login-dashboard-panel branch from 560593d to 6797d04 Compare February 7, 2025 19:16
@StacieClark-Elastic StacieClark-Elastic marked this pull request as ready for review February 7, 2025 19:39
@StacieClark-Elastic StacieClark-Elastic requested a review from a team as a code owner February 7, 2025 19:39
@elastic-vault-github-plugin-prod
Copy link

elastic-vault-github-plugin-prod bot commented Feb 7, 2025

🚀 Benchmarks report

Package google_workspace 👍(10) 💚(3) 💔(2)

Expand to view
Data stream Previous EPS New EPS Diff (%) Result
login 9090.91 7194.24 -1896.67 (-20.86%) 💔
user_accounts 18518.52 13333.33 -5185.19 (-28%) 💔

To see the full report comment with /test benchmark fullreport

Copy link
Contributor

@chemamartinez chemamartinez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Please solve conflicts before merging it.

@elasticmachine
Copy link

💚 Build Succeeded

History

@StacieClark-Elastic StacieClark-Elastic merged commit ff54a12 into elastic:main Feb 11, 2025
6 checks passed
@StacieClark-Elastic StacieClark-Elastic deleted the bugfix/add-missing-filter-to-a-login-dashboard-panel branch February 11, 2025 15:36
@elastic-vault-github-plugin-prod

Package google_workspace - 2.30.4 containing this change is available at https://epr.elastic.co/package/google_workspace/2.30.4/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working, use only for issues Integration:google_workspace Google Workspace Team:Service-Integrations Label for the Observability Service Integrations team

3 participants