🐛 [RUM-97] Sanitize tags parameter in configuration

[RomanGaignault]

Motivation

Setting a non-string value for the customer provided tags init option causes the SDK to fail silently.

Changes

Add validation to ensure that the tags parameters are properly defined as strings.

Testing

• Local
• Staging
• Unit
• End to end

---

I have gone over the contributing documentation.

[bcaudan]

💭 thought: ‏Since we are completely blind on those kind of execution issues, it could worth planning some reflexions around it to see if we could do something better

[cit-pr-commenter]

Bundles Sizes Evolution

📦 Bundle Name	Base Size	Local Size	𝚫	𝚫%	Status
Rum	157.60 KiB	157.78 KiB	184 B	+0.11%	✅
Logs	56.29 KiB	56.47 KiB	184 B	+0.32%	✅
Rum Slim	104.09 KiB	104.27 KiB	184 B	+0.17%	✅
Worker	25.21 KiB	25.21 KiB	0 B	0.00%	✅

🚀 CPU Performance

Action Name	Base Average Cpu Time (ms)	Local Average Cpu Time (ms)	𝚫
adderror	0.030	0.032	0.002
addaction	0.014	0.015	0.001
logmessage	0.005	0.005	0.000
startview	0.804	0.890	0.086
startstopsessionreplayrecording	0.762	1.078	0.316
addtiming	0.001	0.001	0.000
addglobalcontext	0.001	0.001	0.000

[RomanGaignault]

/to-staging

[RomanGaignault]

/to-staging

[dd-devflow]

🚂 Branch Integration: starting soon, merge in < 0s

Commit 5663ef942a will soon be integrated into staging-20.

This build is going to start soon! (estimated merge in less than 0s)

Use /to-staging -c to cancel this operation!

[dd-devflow]

❌ Branch Integration: failure

PR already in the queue with status in_progress

If you need support, contact us on Slack #devflow with those details!

[dd-devflow]

🚂 Branch Integration: This commit was successfully integrated

Commit 5663ef942a has been merged into staging-20 in merge commit 24392cfaf7.

Check out the triggered pipeline on Gitlab 🦊