Skip to content

Conversation

@ShourieG
Copy link
Contributor

@ShourieG ShourieG commented Nov 14, 2025

Type of change

  • Enhancement

Proposed commit message

Sometimes certain fields can be verbose and quite large in size, these often slow down the performance and increase ingestion time. In such scenarios we need an option to filter in/out certain fields directly via query params. This PR makes this behaviour possible. 

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

Author's Checklist

  • [ ]

How to test this PR locally

Related issues

Screenshots

@ShourieG ShourieG self-assigned this Nov 14, 2025
@ShourieG ShourieG added enhancement New feature or request Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations] Integration:qualys_gav Qualys Global AssetView labels Nov 14, 2025
@elastic-vault-github-plugin-prod

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@ShourieG ShourieG marked this pull request as ready for review November 17, 2025 08:36
@ShourieG ShourieG requested a review from a team as a code owner November 17, 2025 08:36
@elasticmachine
Copy link

Pinging @elastic/security-service-integrations (Team:Security-Service Integrations)

Copy link
Contributor

@efd6 efd6 left a comment

Choose a reason for hiding this comment

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

Please add a link to the relevant docs for this.

- name: exclude_fields
type: text
title: Exclude Fields
description: Comma separated list of fields to exclude from the asset object in the response.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
description: Comma separated list of fields to exclude from the asset object in the response.
description: Comma-separated list of fields to exclude from the asset object in the response.
- name: include_fields
type: text
title: Include Fields
description: Comma separated list of fields to be included in the asset object in the response.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
description: Comma separated list of fields to be included in the asset object in the response.
description: Comma-separated list of fields to be included in the asset object in the response.
@elasticmachine
Copy link

💚 Build Succeeded

History

cc @ShourieG

@ShourieG
Copy link
Contributor Author

@efd6, resolved all comments

Copy link
Contributor

@efd6 efd6 left a comment

Choose a reason for hiding this comment

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

Thanks

@ShourieG ShourieG merged commit 7b0e425 into elastic:main Nov 26, 2025
7 checks passed
@elastic-vault-github-plugin-prod

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

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

Labels

enhancement New feature or request Integration:qualys_gav Qualys Global AssetView Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations]

3 participants