[OTTL] Fix empty string matching for replace_all_patterns

[pkcll]

Fix empty string matching for replace_all_patterns, replace_all_matches OTTL functions

Description

OTTL functions replace_all_patterns, replace_all_matches apply pattern matching on all attribute values including non string types by casting value to string using value.Str() method (see here, here) . For non string types Str() returns empty string. In case when empty string patterns is used that creates unintended side effect when pattern matching is applied for non string types.

Example:

• replace_all_patterns(attributes, "value", "^$", "EMPTY_STRING_REPLACEMENT")
• replace_all_matches(attributes, "value", "", "EMPTY_STRING_REPLACEMENT")

Current behavior:

• Replaces all empty string attribute values with "EMPTY_STRING_REPLACEMENT" value.
• Replaces all non string attribute values with "EMPTY_STRING_REPLACEMENT" and converts the type of attributes to string type.

Expected behavior:

• Replaces empty string values in attributes of string type. Skips non string attribute values for matching.

Testing

Added new test case here and here

[linux-foundation-easycla[bot]]

• :x: - login: @pkcll / name: Pavel . The commit (c733e5b1b0396431dd20fb3ed22e6cf9f9afd9db) is not authorized under a signed CLA. Please click here to be authorized. For further assistance with EasyCLA, please submit a support request ticket.

[evan-bradley]

Thanks for opening this @pkcll. I agree this is an issue since replace_pattern and replace_match both check that the target is a string before working with it.

Could you please sign the CLA?

[pkcll]

@evan-bradley

Could you please sign the CLA?

Will do shortly. Let me double check which one should I sign individual CLA or corporate