Skip to content

🐛 [RUM-11614][URL polyfill] Handle null iframe.contentWindow#4361

Open
bcaudan wants to merge 1 commit intomainfrom
bcaudan/url-telemetry
Open

🐛 [RUM-11614][URL polyfill] Handle null iframe.contentWindow#4361
bcaudan wants to merge 1 commit intomainfrom
bcaudan/url-telemetry

Conversation

@bcaudan
Copy link
Collaborator

@bcaudan bcaudan commented Mar 20, 2026

Motivation

Telemetry error:

Cannot read properties of null (reading 'URL')
Cannot read properties of undefined (reading 'URL')
Screenshot 2026-03-20 at 10 44 13

Changes

iframe.contentWindow can be null:

Screenshot 2026-03-20 at 10 45 27

Let's fallback to globalObject in this case.

Test instructions

Let's see if the issue keeps appearing afterwards

Checklist

  • Tested locally
  • Tested on staging
  • Added unit tests for this change.
  • Added e2e/integration tests for this change.
  • Updated documentation and/or relevant AGENTS.md file

@bcaudan bcaudan requested a review from a team as a code owner March 20, 2026 09:47
@datadog-prod-us1-3
Copy link

datadog-prod-us1-3 bot commented Mar 20, 2026

✅ Tests

🎉 All green!

❄️ No new flaky tests detected
🧪 All tests passed

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: cf8d1b0 | Docs | Datadog PR Page | Was this helpful? React with 👍/👎 or give us feedback!

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