Skip to content

AB#129522 - Fix critical / high dependencies in back-end#1231

Merged
AntoineRelief merged 6 commits intonextfrom
AB#129522
Apr 29, 2026
Merged

AB#129522 - Fix critical / high dependencies in back-end#1231
AntoineRelief merged 6 commits intonextfrom
AB#129522

Conversation

@Joselgc1
Copy link
Copy Markdown
Contributor

Description

Updated several backend packages and dependencies to address the npm audit findings, then refreshed the dependency tree by running npm install. It also replaces the jsonpath package with a maintained alternative while keeping the existing usage pattern intact.

Useful links

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • Improvement (refactor or addition to existing functionality)

Checklist:

( * == Mandatory )

  • * I have set myself as assignee of the pull request
  • * My code follows the style guidelines of this project
  • * Linting does not generate new warnings
  • * I have performed a self-review of my own code
  • * I have put the ticket for review, adding the oort-backend team to the list of reviewers
  • * I have commented my code, particularly in hard-to-understand areas
  • * I have put JSDoc comment in all required places
  • * My changes generate no new warnings
  • * I have included screenshots describing my changes if relevant
  • * I have selected labels in the Pull Request, according to the changes with code brings
  • I have made corresponding changes to the documentation ( if required )
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

@Joselgc1 Joselgc1 closed this Apr 27, 2026
@Joselgc1 Joselgc1 reopened this Apr 27, 2026
@Joselgc1 Joselgc1 changed the base branch from main to next April 27, 2026 16:09
@Joselgc1 Joselgc1 changed the title Ab#129522 AB#129522 - Fix critical / high dependencies in back-end Apr 27, 2026
Comment thread package.json
"json2csv": "^5.0.6",
"jsonpath": "^1.1.1",
"jsonwebtoken": "^9.0.0",
"jsonpath-plus": "^10.4.0",
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@Joselgc1
why this change?
jsonpath also has a new version, no?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@AntoineRelief I tried to do that and just upgrade jsonpath to its latest version (1.3.0), but the audit still showed 13 vulnerabilities, with 2 high ones, because jsonpath uses underscore@1.13.6. Then I saw that with jsonpath-plus we had no high errors so I used that. Do you want me to switch it back with the latest version?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@Joselgc1
Okay, well I agree with the change + jsonpath-plus seems to have more support, but let's remove the util & use the library without an additional wrapper

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@Joselgc1
Okay, well I agree with the change + jsonpath-plus seems to have more support, but let's remove the util & use the library without an additional wrapper

@AntoineRelief AntoineRelief merged commit 526a03a into next Apr 29, 2026
1 check passed
@AntoineRelief AntoineRelief deleted the AB#129522 branch April 29, 2026 09:43
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