Skip to content

[FIX] dependabot vulnrability issues#3575

Merged
muli-cohen merged 6 commits into
mlrun:developmentfrom
ilan7empest:ML-12108
Feb 23, 2026
Merged

[FIX] dependabot vulnrability issues#3575
muli-cohen merged 6 commits into
mlrun:developmentfrom
ilan7empest:ML-12108

Conversation

@ilan7empest
Copy link
Copy Markdown
Member

@ilan7empest ilan7empest commented Feb 17, 2026

📝 Description

  1. Fix PostCSS line return parsing error
  2. Fix tmp allows arbitrary temporary file / directory write via symbolic link dir parameter
  3. Fix qs's arrayLimit bypass in comma parsing allows denial of service

🛠️ Changes Made

  • Update dependencies

✅ Checklist

  • I have given the PR a well-structured title describing the domain and the specific change that was made
  • I tested the changes in the browser (locally or via preview build)
  • I confirmed that existing tests pass
  • I added or updated unit / integration tests (if needed)
  • I checked that this change doesn’t introduce new console warnings or lint / formatting errors
  • I updated the relevant Jira ticket with the appropriate details and status

🔗 References


🚨 Potentially Breaking Changes

  • Yes
  • No

Includes DRC change

  • Yes
  • No

If yes -> requires bump NPM version


🔍 Additional Notes


📸 Screenshots / Demos


Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR addresses security vulnerabilities in dependencies by updating three key packages: PostCSS, tmp, and qs. The changes also include a major version update to @cucumber/cucumber (v10 to v12) and various transitive dependency updates reflected in the package-lock.json.

Changes:

  • Updated qs from ^6.14.1 to ^6.14.2 (resolves to 6.15.0) to fix denial of service vulnerability
  • Updated postcss from ^8.4.36 to 8.4.49 (pinned version) to fix line return parsing error
  • Added override for tmp at ^0.2.4 (resolves to 0.2.5) to fix arbitrary file/directory write vulnerability
  • Updated @cucumber/cucumber from ^10.3.1 to 12.6.0 (major version bump with pinned version)
  • Updated package-lock.json with all transitive dependency changes

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated 4 comments.

File Description
package.json Updated dependency versions for qs, postcss, and @cucumber/cucumber; added overrides for qs and tmp
package-lock.json Comprehensive lock file update reflecting all direct and transitive dependency changes

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread package.json Outdated
Comment thread package.json Outdated
Comment thread package.json Outdated
Comment thread package.json Outdated
ilan7empest and others added 5 commits February 17, 2026 10:31
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@muli-cohen muli-cohen changed the title Fix dependabot vulnrability issues [FIX] dependabot vulnrability issues Feb 23, 2026
@muli-cohen muli-cohen merged commit 85decd2 into mlrun:development Feb 23, 2026
3 checks passed
@ilan7empest ilan7empest deleted the ML-12108 branch February 24, 2026 08:36
Taras-Hlukhovetskyi pushed a commit to Taras-Hlukhovetskyi/ui that referenced this pull request Feb 24, 2026
  ## 📝 Description
  <!-- A clear, concise summary of what this PR does. -->
  <!-- Include context, motivation, or related issues (e.g., "Replaces Sass with CSS Modules"). -->

1. Fix PostCSS line return parsing error
2. Fix tmp allows arbitrary temporary file / directory write via symbolic link `dir` parameter
3. Fix qs's arrayLimit bypass in comma parsing allows denial of service
---

## 🛠️ Changes Made
  <!-- List the main updates in this PR. -->
<!-- Example:
  - Replaced Sass styles with CSS Modules
  - Updated Button component props for accessibility
  - Adjusted CI workflow to run on push for all branches
  -->
 - Update dependencies
---

## ✅ Checklist

- [x] I have given the PR a well-structured title describing the domain and the specific change that was made
- [x] I tested the changes in the browser (locally or via preview build)
- [x] I confirmed that existing tests pass
- [ ] I added or updated unit / integration tests (if needed)
- [x] I checked that this change doesn’t introduce new console warnings or lint / formatting errors
- [ ] I updated the relevant Jira ticket with the appropriate details and status


---

## 🔗 References
- Related ticket / issue: [ML-12108](https://iguazio.atlassian.net/browse/ML-12108), [ML-12109](https://iguazio.atlassian.net/browse/ML-12109), [ML-12170](https://iguazio.atlassian.net/browse/ML-12170)
- Figma / design spec:
- Documentation:

---

## 🚨 Potentially Breaking Changes
- [ ] Yes
- [ ] No

<!-- If yes, describe what breaks and how to migrate: -->
<!-- Example: Renamed prop `variant` → `type` in Button component -->

---

Includes DRC change
- [ ] Yes
- [x] No

If yes -> requires bump NPM version

---

## 🔍 Additional Notes
<!-- Optional: other context, performance considerations, or follow-up work -->
<!-- Example:
  - TODO: Migrate remaining components away from Sass
  - Known issue: minor layout flicker on mobile
  -->

---

## 📸 Screenshots / Demos
<!-- Include before/after screenshots, GIFs, or video demos if the change affects UI. -->
<!-- You can drag and drop images here directly in the PR. -->

---


[ML-12108]: https://iguazio.atlassian.net/browse/ML-12108?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ
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