Skip to content

Fix security vulnerabilities by upgrading dependencies#3616

Open
abhandage wants to merge 2 commits intomainfrom
fix/upgrade-vulnerable-dependencies
Open

Fix security vulnerabilities by upgrading dependencies#3616
abhandage wants to merge 2 commits intomainfrom
fix/upgrade-vulnerable-dependencies

Conversation

@abhandage
Copy link
Contributor

  • Upgrade @aws-sdk/client-s3 from ^3.600.0 to ^3.894.0
  • Upgrade @aws-sdk/client-sts from 3.614.0 to ^3.894.0
  • Upgrade ajv from ^8.6.3 to ^8.18.0

This resolves the vulnerable fast-xml-parser@4.2.5 transitive dependency from older AWS SDK versions and addresses ajv security issues.

Testing

Include any additional information about the testing you have completed to
ensure your changes behave as expected. For a speedy review, please check
any of the tasks you completed below during your testing.

  • Added unit tests for new functionality
  • Tested end-to-end using the local server
  • [If destination is already live] Tested for backward compatibility of destination. Note: New required fields are a breaking change.
  • [Segmenters] Tested in the staging environment
  • [Segmenters] [If applicable for this change] Tested for regression with Hadron.

Security Review

Please ensure sensitive data is properly protected in your integration.

  • Reviewed all field definitions for sensitive data (API keys, tokens, passwords, client secrets) and confirmed they use type: 'password'

New Destination Checklist

  • Extracted all action API versions to verioning-info.ts file. example

- Upgrade @aws-sdk/client-s3 from ^3.600.0 to ^3.894.0
- Upgrade @aws-sdk/client-sts from 3.614.0 to ^3.894.0
- Upgrade ajv from ^8.6.3 to ^8.18.0

This resolves the vulnerable fast-xml-parser@4.2.5 transitive dependency
from older AWS SDK versions and addresses ajv security issues.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Update test to properly mock @aws-sdk/client-sts v3.894.0 by using
jest.mock() factory functions and importing mocked modules at the top.
This ensures compatibility with the upgraded AWS SDK.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@codecov
Copy link

codecov bot commented Feb 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.71%. Comparing base (c5c75a3) to head (cd69093).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3616      +/-   ##
==========================================
+ Coverage   80.34%   80.71%   +0.37%     
==========================================
  Files        1283     1593     +310     
  Lines       23583    30517    +6934     
  Branches     4728     6592    +1864     
==========================================
+ Hits        18947    24632    +5685     
- Misses       3755     4994    +1239     
- Partials      881      891      +10     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@abhandage abhandage marked this pull request as ready for review February 20, 2026 09:25
@abhandage abhandage requested a review from a team as a code owner February 20, 2026 09:25
"license": "MIT",
"devDependencies": {
"ajv": "^8.6.3",
"ajv": "^8.18.0",
Copy link
Contributor

Choose a reason for hiding this comment

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

Just want to make sure this bump doesn't impact any functionalities. Since this is a minor version bump, not expecting many surprises. But it would good to look a the change long once and confirming

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments