Skip to content

Conversation

@pkaeding
Copy link
Contributor

@pkaeding pkaeding commented Dec 3, 2025

Summary

Fixes SEC-7530 by adding a yarn resolution to override the vulnerable transitive dependency react-server-dom-webpack from version 19.0.0-rc-6230622a1a-20240610 to the safe version 19.0.1.

The vulnerable version was pulled in by jest-expo@~52.0.2. Using a yarn resolution is the standard approach to override transitive dependencies.

Link to Devin run: https://app.devin.ai/sessions/b4c805fbfef942e1adbe1b06a11d5f3c
Requested by: Patrick Kaeding (@pkaeding)

How did you test this change?

  • Ran yarn install to verify the resolution is applied correctly
  • Ran yarn format:all to ensure code formatting passes
  • CI will validate that tests pass with the updated dependency

Are there any deployment considerations?

No deployment considerations. This is a dev/test dependency update only.


Note

Pins react-server-dom-webpack to 19.0.1 with a Yarn resolution and updates lockfile (peer ranges and webpack-sources).

  • Dependencies:
    • Add Yarn resolutions entry to force [email protected] in package.json.
    • Update yarn.lock to resolve react-server-dom-webpack to 19.0.1 with updated peer deps (react, react-dom -> ^19.0.1).
    • Add webpack-sources@^3.2.0 dependency (resolved to 3.3.3) required by the updated package.

Written by Cursor Bugbot for commit dbecbbb. This will update automatically on new commits. Configure here.


Related Jira issue: SEC-7530: Update react-server-dom-webpack in observability-sdk

Add yarn resolution to override the vulnerable transitive dependency
react-server-dom-webpack from version 19.0.0-rc-6230622a1a-20240610
to the safe version 19.0.1.

The vulnerable version was pulled in by jest-expo@~52.0.2.

Co-Authored-By: Patrick Kaeding <[email protected]>
@pkaeding pkaeding requested a review from a team as a code owner December 3, 2025 21:39
@devin-ai-integration
Copy link
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

Run yarn dedupe to consolidate webpack-sources versions as required
by CI's dedupe check.

Co-Authored-By: Patrick Kaeding <[email protected]>
@pkaeding pkaeding requested a review from a team December 3, 2025 21:54
Copy link

@mayberryzane mayberryzane left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@pkaeding pkaeding merged commit 4ad4604 into main Dec 4, 2025
23 checks passed
@pkaeding pkaeding deleted the devin/1764797890-sec-7530-update-react-server-dom-webpack branch December 4, 2025 02:06
abelonogov-ld added a commit that referenced this pull request Dec 4, 2025
* main:
  chore: release main (#307)
  fix(SEC-7530): update react-server-dom-webpack to 19.0.1 (#310)
abelonogov-ld added a commit that referenced this pull request Dec 4, 2025
* main:
  doc: Add using ldMask in readme. (#311)
  chore: release main (#312)
  feat: take transformed coordinates, which are more precise in animation  (#309)
  chore: release main (#307)
  fix(SEC-7530): update react-server-dom-webpack to 19.0.1 (#310)

# Conflicts:
#	sdk/@launchdarkly/observability-android/lib/src/main/kotlin/com/launchdarkly/observability/replay/capture/CaptureSource.kt
#	sdk/@launchdarkly/observability-android/lib/src/main/kotlin/com/launchdarkly/observability/replay/masking/ComposeMaskTarget.kt
#	sdk/@launchdarkly/observability-android/lib/src/main/kotlin/com/launchdarkly/observability/replay/masking/Mask.kt
#	sdk/@launchdarkly/observability-android/lib/src/main/kotlin/com/launchdarkly/observability/replay/masking/NativeMaskTarget.kt
abelonogov-ld added a commit that referenced this pull request Dec 4, 2025
* main: (85 commits)
  doc: Add using ldMask in readme. (#311)
  chore: release main (#312)
  feat: take transformed coordinates, which are more precise in animation  (#309)
  chore: release main (#307)
  fix(SEC-7530): update react-server-dom-webpack to 19.0.1 (#310)
  feat: recursive mask collection (#308)
  feat: support non-standard windows added by WindowManager (#306)
  feat: Android SR Do not send duplicate screens (#304)
  test: Add UI and logic to evaluate boolean flags (#305)
  chore: release main (#303)
  feat: Android Dialog Capture (#302)
  chore: Update Android Test main screen (#301)
  chore: release main (#300)
  feat: XML Views Automasking options (#299)
  fix: missed imports (#298)
  chore: release main (#297)
  feat: Support ldMask() for Native and Compose views. (#295)
  chore: release main (#296)
  fix: tweaks Android InteractionDetector to delegate additional defaul… (#294)
  chore: release main (#293)
  ...

# Conflicts:
#	e2e/react-router/src/ldclientLazy.tsx
#	e2e/react-router/src/routes/root.tsx
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.

3 participants