Skip to content

[Snyk] Security upgrade react-player from 2.11.0 to 3.3.1 - #522

Open
snyk-io[bot] wants to merge 1 commit into
masterfrom
snyk-fix-5da49912840951005e35f5d4144cd086
Open

snyk-io[bot] wants to merge 1 commit into
masterfrom
snyk-fix-5da49912840951005e35f5d4144cd086

Conversation

@snyk-io

@snyk-io snyk-io Bot commented Sep 20, 2026

Copy link
Copy Markdown

snyk-top-banner

Snyk has created this PR to fix 1 vulnerabilities in the npm dependencies of this project.

Snyk changed the following file(s):

  • package.json
  • package-lock.json

Vulnerabilities that will be fixed with an upgrade:

Issue Score
high severity Prototype Pollution
SNYK-JS-DEEPMERGE-19964053
  721  

Breaking Change Risk

Merge Risk: High

Notice: This assessment is enhanced by AI.


Important

  • Check the changes in this PR to ensure they won't cause issues with your project.
  • Max score is 1000. Note that the real score may have changed since the PR was raised.
  • This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.

For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic


Learn how to fix vulnerabilities with free interactive lessons:

🦉 Prototype Pollution

Copilot AI lite review requested due to automatic review settings September 20, 2026 16:39
@snyk-io

snyk-io Bot commented Sep 20, 2026

Copy link
Copy Markdown
Author

Merge Risk: High

The upgrade from react-player v2 to v3 is a major version with significant breaking changes, as it is a complete rewrite of the library and is not backwards compatible.

Key Breaking Changes in v3.0.0:

  • Complete Rewrite: The library has been rewritten from the ground up using TypeScript and React function components, which may require significant refactoring of how the component is used.
  • Build System Changes: Support for CJS (require()) and IIFE (<script> tag) bundles has been removed. You must use ES Modules (import) to use the library.
  • Removed Media Providers: Several media providers are no longer included in the base package: Dailymotion, Facebook, Kaltura, MixCloud, SoundCloud, Streamable, Twitch, and Vidyard.
  • React Suspense: Suspense is now enabled by default.

Source: GitHub Releases

Recommendation: This upgrade will require code modifications. Developers must review the official migration guide, update their import statements, ensure they are not using a removed media provider, and test thoroughly to adapt to the new component architecture.

Notice 🤖: This content was augmented using artificial intelligence. AI-generated content may contain errors and should be reviewed for accuracy before use.

@snyk-io

snyk-io Bot commented Sep 20, 2026 •

Copy link
Copy Markdown
Author

✅ Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
✅ Open Source Security 0 0 0 0 0 issues
✅ Licenses 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

Critical compatibility issues and stale license metadata remain unresolved.

Get a fresh assessment by requesting another Copilot review.

Review effort: Lite
Findings: 2 High severity · 1 Low severity

Open (3)
What changed in this PR

Upgrades react-player from 2.11.0 to 3.3.1 to remediate a prototype-pollution vulnerability.

Changes:

  • Updates the direct react-player dependency.
  • Refreshes the lockfile dependency tree and removes deepmerge.
File Summary
package.json Dependency upgrade. Unresolved critical findings: update media call sites from url to src (3 votes) and align React type packages (1 vote). Nit: regenerate OSS_LICENSES.txt (2 votes).
package-lock.json Locks the new dependency tree. Moderate finding: the React type peer dependency is incompatible with the current lockfile (2 votes).

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread package.json
"react-animate-height": "^2.0.23",
"react-dom": "^17.0.2",
"react-player": "^2.9.0",
"react-player": "^3.3.1",
Comment thread package.json
"react-animate-height": "^2.0.23",
"react-dom": "^17.0.2",
"react-player": "^2.9.0",
"react-player": "^3.3.1",
Comment thread package.json
"react-animate-height": "^2.0.23",
"react-dom": "^17.0.2",
"react-player": "^2.9.0",
"react-player": "^3.3.1",
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.

1 participant