chore(deps): update peerdependency axios to ^0.28.0 [security]#45
chore(deps): update peerdependency axios to ^0.28.0 [security]#45renovate[bot] wants to merge 1 commit intodevelopfrom
Conversation
|
|
Important Review skippedBot user detected. To trigger a single review, invoke the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
📝 WalkthroughWalkthroughThis update primarily involves an enhancement to the Changes
Poem
✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 0
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- package.json (1 hunks)
Additional comments not posted (1)
package.json (1)
54-55: LGTM! But verify for breaking changes.The update to
axiosversion^0.28.0addresses a security vulnerability and includes several improvements.However, ensure that this update does not introduce breaking changes in the codebase.
Verification successful
Verification Successful: No Breaking Changes Found
The update to
axiosversion^0.28.0does not introduce breaking changes in the codebase. Thepostmethod used in the code remains compatible with the new version.
test/index.test.js: Mockingaxios.postsrc/index.js: Usingaxios.postto make a POST requestScripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify that the update to axios version ^0.28.0 does not introduce breaking changes. # Test: Search for axios usage in the codebase. Expect: No breaking changes. rg --type js --type ts --type vue 'axios'Length of output: 69
Script:
#!/bin/bash # Description: Verify that the update to axios version ^0.28.0 does not introduce breaking changes. # Test: Search for axios usage in the codebase. Expect: No breaking changes. rg --type js --type ts --type jsx --type tsx 'axios'Length of output: 80
Script:
#!/bin/bash # Description: Verify that the update to axios version ^0.28.0 does not introduce breaking changes. # Test: Search for axios usage in the codebase. Expect: No breaking changes. rg --type js --type ts 'axios'Length of output: 174
Script:
#!/bin/bash # Extract relevant code segments where axios is used for inspection. # Extract from test/index.test.js echo "test/index.test.js:" rg 'axios' test/index.test.js -A 5 # Extract from src/index.js echo "src/index.js:" rg 'axios' src/index.js -A 5Length of output: 656
c2f985c to
af47417
Compare
af47417 to
2faac7b
Compare
f39c25c to
f8cdbca
Compare
f8cdbca to
0e11e9e
Compare
0e11e9e to
01e763f
Compare
01e763f to
8f18a4e
Compare
8f18a4e to
61f31c0
Compare
61f31c0 to
e8dec8c
Compare
e8dec8c to
da364b3
Compare
da364b3 to
54a96d1
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #45 +/- ##
========================================
Coverage 95.77% 95.77%
========================================
Files 1 1
Lines 71 71
Branches 27 27
========================================
Hits 68 68
Misses 1 1
Partials 2 2 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
54a96d1 to
793730d
Compare
79e3dd5 to
cd3e094
Compare
8651bc3 to
35cb3d2
Compare
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
package.json (1)
53-56: Dependency Update: Verify New Axios Version CompatibilityThe peer dependency for Axios has been updated to
"^1.8.2", which does not align with the PR title and objectives indicating an update to"^0.28.0". Please confirm that this major version upgrade is intentional and that downstream consumers are compatible with Axios 1.x. Additionally, consider updating the documentation or migration guides to highlight any breaking changes introduced by this version change.
9444b3d to
b0cce34
Compare
1ad3577 to
85a22e0
Compare
85a22e0 to
fe7bf09
Compare
fe7bf09 to
0972a1c
Compare
0972a1c to
24909ef
Compare
edef26c to
a094e5c
Compare
a094e5c to
6c55b43
Compare
6c55b43 to
5818669
Compare
This PR contains the following updates:
^0.27.2→^0.28.0GitHub Vulnerability Alerts
CVE-2023-45857
An issue discovered in Axios 0.8.1 through 1.5.1 inadvertently reveals the confidential XSRF-TOKEN stored in cookies by including it in the HTTP header X-XSRF-TOKEN for every request made to any host allowing attackers to view sensitive information.
Release Notes
axios/axios (axios)
v0.28.0Compare Source
Release notes:
Bug Fixes
withXSRFTokenoption to v0.x (#6091)Backports from v1.x:
axios.formToJSONmethod (#4735)url-encoded-formserializer to respect theformSerializerconfig (#4721)string[]toAxiosRequestHeaderstype (#4322)AxiosErrorstack capturing; (#4718)AxiosErrorstatus code type; (#4717)blobto the list of protocols supported by the browser (#4678)Configuration
📅 Schedule: Branch creation - "" (UTC), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.