Skip to content

Conversation

sbarrio
Copy link
Contributor

@sbarrio sbarrio commented Aug 13, 2025

What does this PR do?

We have new security warnings raised by dependabot:

https://github.com/DataDog/dd-sdk-reactnative/security/dependabot/42
https://github.com/DataDog/dd-sdk-reactnative/security/dependabot/30

Also, we have a typo on the resolution for fast-xml-parser.

Motivation

We want to keep dependencies up to date and set to secure versions.

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests
  • Make sure you discussed the feature or bugfix with the maintaining team in an Issue
  • Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)
  • If this PR is auto-generated, please make sure also to manually update the code related to the change

@sbarrio sbarrio requested a review from a team as a code owner August 13, 2025 09:14
@@ -89,14 +89,15 @@
"ejs": "3.1.10",
"shell-quote": "^1.7.3",
"cross-spawn": "7.0.5",
"ft-xml-parser": "4.4.1",
"fast-xml-parser": "4.4.1",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@sbarrio sbarrio self-assigned this Aug 20, 2025
@sbarrio sbarrio merged commit d16ff47 into develop Aug 21, 2025
10 checks passed
@sbarrio sbarrio deleted the sbarrio/fix/solve-security-dependencies-for-fast-xml-parser-tmp-and-babel-runtime branch August 21, 2025 07:25
@@ -34,7 +34,7 @@
"@babel/core": "^7.25.2",
"@babel/plugin-transform-runtime": "^7.25.0",
"@babel/preset-env": "^7.25.3",
"@babel/runtime": "^7.25.0",
"@babel/runtime": "7.26.10",
Copy link
Member

Choose a reason for hiding this comment

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

Should use a caret here for compatibility

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