[exporter/elasticsearch] Add sanitization utils for datastream fields

[rubvs]

Description: Adds util functions to sanitize data stream fields

Link to tracking Issue: Fixes #34285

Testing: Added unit tests

[linux-foundation-easycla]

The committers listed above are authorized under a signed CLA.

• ✅ login: rubvs / name: Ruben van Staden (bc3bd6b, e973b09, 4383f67, d69cd4a, f30ab50, fe77f13, 188300c, bf248b3, 86cc2ae, f01217c, 8bbbc8c, 9e3564e, 8bdea23, 4bafdcf, 71ffaca, 5d771ce, a868f0a, d89ed94, ce54d20, 4020fc7, 43e55d2, f85b3a5)

[carsonip]

Thanks, looks good functionality wise. A couple of minor comments.

[rubvs]

@carsonip I have improved the sanitizeDataStreamField to leverage inlining. I also merged the .otel suffix extension into the sanitization check, since I have to take the extension into account when doing the 100 char check. Let me know if you approve this approach.

[rubvs]

@carsonip I also added a test to model_test.go. I do not know if this is needed, but I think it won't hurt to add it either. If you want me to keep this, I'll update the test to get rid of the disallowedRunes and replace if with the hardcoded runes, as you suggested in the previous comments.

[carsonip]

nice catch about the otel suffix! a comment about refactoring

[andrzej-stencel]

Shouldn't this be documented?

[rubvs]

Shouldn't this be documented?

Hi @andrzej-stencel, I've updated the readme of ES exported to reference the restriction docs. Let me know if this suffices, or if you think I should articulate things differently.

[andrzej-stencel]

Shouldn't this be documented?

I've updated the readme of ES exporter to reference the restriction docs. Let me know if this suffices, or if you think I should articulate things differently.

Thanks @rubvs, this is better than nothing. Ideally the docs would mention that the restricted characters will be replaced with underscores _.

[atoulme]

Please resolve conflicts.