Skip to content

✅ improve format validation error output#3834

Merged
BenoitZugmeyer merged 1 commit intomainfrom
benoit/push-ywtznoytrsqn
Sep 11, 2025
Merged

✅ improve format validation error output#3834
BenoitZugmeyer merged 1 commit intomainfrom
benoit/push-ywtznoytrsqn

Conversation

@BenoitZugmeyer
Copy link
Member

@BenoitZugmeyer BenoitZugmeyer commented Sep 11, 2025

Motivation

When validating the RUM events schema, the error produced by additional properties isn't very informative

Changes

Add which property is unexpected in the error message

Test instructions

Checklist

  • Tested locally
  • Tested on staging
  • Added unit tests for this change.
  • Added e2e/integration tests for this change.

@BenoitZugmeyer BenoitZugmeyer requested a review from a team as a code owner September 11, 2025 13:46
@datadog-datadog-prod-us1
Copy link

✅ Tests

🎉 All green!

❄️ No new flaky tests detected
🧪 All tests passed

🎯 Code Coverage
Patch Coverage: 0.00%
Total Coverage: 92.25% (-0.03%)

View detailed report

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 6817a40 | Docs | Was this helpful? Give us feedback!

@cit-pr-commenter
Copy link

Bundles Sizes Evolution

📦 Bundle Name Base Size Local Size 𝚫 𝚫% Status
Rum 157.25 KiB 157.25 KiB 0 B 0.00%
Rum Recorder 19.53 KiB 19.53 KiB 0 B 0.00%
Rum Profiler 5.18 KiB 5.18 KiB 0 B 0.00%
Logs 55.74 KiB 55.74 KiB 0 B 0.00%
Flagging N/A 931 B 931 B N/A%
Rum Slim 114.99 KiB 114.99 KiB 0 B 0.00%
Worker 23.60 KiB 23.60 KiB 0 B 0.00%
🚀 CPU Performance
Action Name Base Average Cpu Time (ms) Local Average Cpu Time (ms) 𝚫
addglobalcontext 0.008 0.008 -0.001
addaction 0.029 0.018 -0.011
addtiming 0.009 0.004 -0.005
adderror 0.031 0.021 -0.010
startstopsessionreplayrecording 0.003 0.001 -0.001
startview 0.011 0.005 -0.007
logmessage 0.057 0.024 -0.033
🧠 Memory Performance
Action Name Base Consumption Memory (bytes) Local Consumption Memory (bytes) 𝚫 (bytes)
addglobalcontext 25.77 KiB 26.27 KiB 513 B
addaction 45.93 KiB 46.07 KiB 145 B
addtiming 24.96 KiB 24.09 KiB -895 B
adderror 49.96 KiB 51.33 KiB 1.37 KiB
startstopsessionreplayrecording 24.03 KiB 24.10 KiB 66 B
startview 424.71 KiB 428.72 KiB 4.01 KiB
logmessage 42.64 KiB 42.43 KiB -213 B

🔗 RealWorld

@BenoitZugmeyer BenoitZugmeyer merged commit c7a22e1 into main Sep 11, 2025
21 checks passed
@BenoitZugmeyer BenoitZugmeyer deleted the benoit/push-ywtznoytrsqn branch September 11, 2025 15:35
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.

2 participants