fix: Revert "fix: update tmp dependency to solve CVE-2026-44705"#6875
Conversation
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
8f06c26 to
17a1d7f
Compare
This comment has been minimized.
This comment has been minimized.
17a1d7f to
58c0fb7
Compare
This comment has been minimized.
This comment has been minimized.
58c0fb7 to
1caf347
Compare
This comment has been minimized.
This comment has been minimized.
1caf347 to
bc2ca04
Compare
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
@robertolopezlopez Why is this diff so big? Did you use a different NodeJs version?
There was a problem hiding this comment.
CI also has a few weird failures, I think something is not right.
There was a problem hiding this comment.
There was a problem hiding this comment.
The big diff may be related to the version bump for @snyk/snyk-hex-plugin and snyk-go-plugin
There was a problem hiding this comment.
| "minimatch@^3.1.2": "3.1.3", | ||
| "tmp": "0.2.7", | ||
| "snyk-python-plugin": { | ||
| "tmp@<0.2.6": "^0.2.6" |
There was a problem hiding this comment.
I am adding this override here until snyk-python-plugin will be fixed upstream
|
ok
…On Wed, Jun 3, 2026 at 4:31 PM CatalinSnyk ***@***.***> wrote:
***@***.**** commented on this pull request.
------------------------------
On package-lock.json
<#6875 (comment)>:
Could you try to regenerate this again? I did the same changes locally to
the package.json file and I got this diff for the lockfile, which I think
makes more sense. The +7.8k -5.5k sounds too big for just 2-3 dependency
changes (also most of them are minor updates).
image.png (view on web)
<https://github.com/user-attachments/assets/04bb3fab-2957-417f-bbf8-1b49772d25c4>
—
Reply to this email directly, view it on GitHub
<#6875?email_source=notifications&email_token=AALS624T7V4NCV4XFG2NI2T46AZETA5CNFSNUABKM5UWIORPF5TWS5BNNB2WEL2QOVWGYUTFOF2WK43UKJSXM2LFO4XTINBRHE2TAOJVHA42M4TFMFZW63VHNVSW45DJN5XKKZLWMVXHJLDGN5XXIZLSL5RWY2LDNM#discussion_r3349292834>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AALS623LT25MC5B3KAW35OL46AZETAVCNFSM6AAAAACZXMA6N2VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHM2DIMJZGUYDSNJYHE>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
bacb7fe to
af07bad
Compare
This comment has been minimized.
This comment has been minimized.
af07bad to
69eddd0
Compare
This comment has been minimized.
This comment has been minimized.
5ccdc5d to
aae1f65
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
17e70ca to
8f5680b
Compare
8f5680b to
3094112
Compare
This comment has been minimized.
This comment has been minimized.
PR Reviewer Guide 🔍
|




Pull Request Submission Checklist
are release-note ready, emphasizing
what was changed, not how.
What does this PR do?
overridefrom previous merge.tmppackagesnyk-python-pluginwhich installstmp@0.2.6in case this dependency is affected by CVE-2026-44705Where should the reviewer start?
How should this be manually tested?
What's the product update that needs to be communicated to CLI users?