Skip to content

Wrong type for lookup.default_match#371

Merged
pyth0n1c merged 5 commits intomainfrom
fix_default_match_type
Mar 10, 2025
Merged

Wrong type for lookup.default_match#371
pyth0n1c merged 5 commits intomainfrom
fix_default_match_type

Conversation

@pyth0n1c
Copy link
Copy Markdown
Contributor

@pyth0n1c pyth0n1c commented Feb 13, 2025

Copy link
Copy Markdown
Contributor Author

@pyth0n1c pyth0n1c left a comment

Choose a reason for hiding this comment

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

transforms.conf is identical after the change
image

This also fixes an issue where if default_match was not defined, it would be written as "false" in any api objects with a serialized lookup. For instance:
image

Because this is now always treated as a string (not a bool) if this field is not defined then it is an empty string, which is the default value for this field:
https://docs.splunk.com/Documentation/Splunk/9.4.1/Admin/Transformsconf#:~:text=provided%20(see%20below).-,default_match,-%3D%20%3Cstring%3E%0A*%20If%20min_matches

@pyth0n1c pyth0n1c merged commit 46378f7 into main Mar 10, 2025
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant