Skip to content

fix: pin axios to 1.13.2 to prevent upgrade to compromised versions #3864

Open
priyanshu6238 wants to merge 1 commit intomasterfrom
fix/axois_version
Open

fix: pin axios to 1.13.2 to prevent upgrade to compromised versions #3864
priyanshu6238 wants to merge 1 commit intomasterfrom
fix/axois_version

Conversation

@priyanshu6238
Copy link
Copy Markdown
Collaborator

@priyanshu6238 priyanshu6238 commented Apr 1, 2026

Summary

Test Plan

Summary by CodeRabbit

  • Chores
    • Updated dependency version specification to ensure consistent package resolution.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 1, 2026

Walkthrough

The pull request modifies the package.json file, changing the axios dependency version specification from a caret-pinned range (^1.13.2) to an exact version pin (1.13.2). This constrains npm/yarn to install precisely version 1.13.2 instead of allowing minor or patch version upgrades within the range. No other dependencies, scripts, or configuration values were altered.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A rabbit hops through version land,
Where caret signs were once so grand,
But now we pin with steady care—
One-dot-three-dot-two, declared with flair! 📌

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning The title mentions pinning axios to prevent upgrades to compromised versions, but the actual change simply removes the caret operator (^) from the version spec without addressing security vulnerabilities. Update the title to accurately describe the change: 'fix: pin axios to exact version 1.13.2' or clarify in the PR description why this specific version prevents compromised upgrades.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/axois_version

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 1, 2026

@github-actions github-actions bot temporarily deployed to pull request April 1, 2026 01:24 Inactive
@codacy-production
Copy link
Copy Markdown

codacy-production bot commented Apr 1, 2026

Not up to standards ⛔

🔴 Issues 1 high

Alerts:
⚠ 1 issue (≤ 0 issues of at least minor severity)

Results:
1 new issue

Category Results
Security 1 high

View in Codacy

🟢 Metrics 0 complexity

Metric Results
Complexity 0

View in Codacy

TIP This summary will be updated as you push new changes. Give us feedback

@priyanshu6238 priyanshu6238 changed the title fix: remove the range symbol from axois fix: pin axios to 1.13.2 to prevent upgrade to compromised versions Apr 1, 2026
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 1, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.88%. Comparing base (c56b13f) to head (1af90d8).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #3864   +/-   ##
=======================================
  Coverage   81.88%   81.88%           
=======================================
  Files         308      308           
  Lines       12940    12940           
  Branches     2978     2978           
=======================================
  Hits        10596    10596           
  Misses       1409     1409           
  Partials      935      935           

☔ 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.

@cypress
Copy link
Copy Markdown

cypress bot commented Apr 1, 2026

Glific    Run #8084

Run Properties:  status check failed Failed #8084  •  git commit 00aad3933a ℹ️: Merge 1af90d876e5505b97d6d34773c003f53cdeeb490 into c56b13f7ed7fa1dbac09a9b51dc9...
Project Glific
Branch Review fix/axois_version
Run status status check failed Failed #8084
Run duration 13m 05s
Commit git commit 00aad3933a ℹ️: Merge 1af90d876e5505b97d6d34773c003f53cdeeb490 into c56b13f7ed7fa1dbac09a9b51dc9...
Committer Priyanshu singh
View all properties for this run ↗︎

Test results
Tests that failed  Failures 1
Tests that were flaky  Flaky 1
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 97
View all changes introduced in this branch ↗︎

Tests for review

Failed  cypress/e2e/filesearch/Filesearch.spec.ts • 1 failed test

View Output

Test Artifacts
File search > should create a new assistant with file upload Test Replay Screenshots
Flakiness  cypress/e2e/chat/ChatCollection.spec.ts • 1 flaky test

View Output

Test Artifacts
ChatCollection > should send the emoji to collection Test Replay Screenshots

@priyanshu6238 priyanshu6238 self-assigned this Apr 1, 2026
Copy link
Copy Markdown
Contributor

@rvignesh89 rvignesh89 left a comment

Choose a reason for hiding this comment

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

Don't think we need to pin to exact version since the vulnerable packages have been removed from npmjs

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