Skip to content

[Tenable IO] Drop empty events in vulnerability data stream - #17878

Merged
moxarth-rathod merged 5 commits into
elastic:mainfrom
moxarth-rathod:tenable-io-17608
Mar 30, 2026
Merged

[Tenable IO] Drop empty events in vulnerability data stream#17878
moxarth-rathod merged 5 commits into
elastic:mainfrom
moxarth-rathod:tenable-io-17608

Conversation

@moxarth-rathod

Copy link
Copy Markdown
Contributor

Proposed commit message

tenable_io: drop empty events in vulnerability data stream

The CEL program emits empty events ([{}]) on two fallback paths, 
causing the error field [original] not present as part of path [event.original]. 
Add a drop-event processor in CEL to discard empty events.

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

Related issues

@moxarth-rathod moxarth-rathod self-assigned this Mar 18, 2026
@moxarth-rathod moxarth-rathod added the Integration:tenable_io Tenable Vulnerability Management label Mar 18, 2026
@moxarth-rathod
moxarth-rathod requested a review from a team as a code owner March 18, 2026 12:04
@moxarth-rathod moxarth-rathod added bugfix Pull request that fixes a bug issue Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations] Team:SDE-Crest Crest developers on the Security Integrations team [elastic/sit-crest-contractors] labels Mar 18, 2026
@elasticmachine

Copy link
Copy Markdown

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

@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@efd6 efd6 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can we make this more explicit? Instead of using an empty sentinel, use [{"retry": true}] and then have - drop_event.when.equals.retry: true in the processors. This makes it easier to see the intent and to find the locations.

@moxarth-rathod
moxarth-rathod requested a review from efd6 March 20, 2026 05:43
Comment thread packages/tenable_io/data_stream/vulnerability/agent/stream/cel.yml.hbs Outdated
Comment thread packages/tenable_io/data_stream/vulnerability/agent/stream/cel.yml.hbs Outdated
Comment thread packages/tenable_io/data_stream/vulnerability/agent/stream/cel.yml.hbs Outdated
@moxarth-rathod
moxarth-rathod requested a review from efd6 March 26, 2026 06:54
Comment thread packages/tenable_io/data_stream/vulnerability/agent/stream/cel.yml.hbs Outdated
@moxarth-rathod
moxarth-rathod requested a review from kcreddy March 26, 2026 08:29
@elasticmachine

Copy link
Copy Markdown

💚 Build Succeeded

History

cc @moxarth-rathod

@kcreddy kcreddy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM. Please wait for @efd6

@efd6 efd6 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks

@moxarth-rathod
moxarth-rathod merged commit 015d2fe into elastic:main Mar 30, 2026
11 checks passed
@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

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

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

Labels

bugfix Pull request that fixes a bug issue Integration:tenable_io Tenable Vulnerability Management Team:SDE-Crest Crest developers on the Security Integrations team [elastic/sit-crest-contractors] Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations]

4 participants