Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(transforms): Fix matching boolean values for Datadog search s… #21624

Merged

Conversation

ArunPiduguDD
Copy link
Contributor

Overview

There is a bug in the current implementation of Datadog search syntax which causes queries based on attributes with boolean values to be ignored. This is b/c we are only checking if the type of the attribute is a string or numeric (currently ignoring the boolean case).

Fixes this bug by checking if the attribute value is type boolean

@ArunPiduguDD ArunPiduguDD requested a review from a team as a code owner October 25, 2024 21:16
@ArunPiduguDD ArunPiduguDD changed the title fix(datadog_search): Fix matching boolean values for Datadog search s… fix(transforms): Fix matching boolean values for Datadog search s… Oct 25, 2024
@bruceg bruceg added type: bug A code related bug. provider: datadog Anything `datadog` service provider related labels Oct 25, 2024
Copy link
Member

@bruceg bruceg left a comment

Choose a reason for hiding this comment

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

A couple of nit-picks about wording but otherwise LGTM.

src/conditions/datadog_search.rs Outdated Show resolved Hide resolved
src/conditions/datadog_search.rs Outdated Show resolved Hide resolved
Copy link
Member

@pront pront left a comment

Choose a reason for hiding this comment

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

Let's add a changelog

@pront pront added this pull request to the merge queue Oct 28, 2024
Merged via the queue into vectordotdev:master with commit 4c3aa35 Oct 28, 2024
40 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
provider: datadog Anything `datadog` service provider related type: bug A code related bug.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants