Skip to content

Conversation

@matz3
Copy link
Member

@matz3 matz3 commented Jan 23, 2026

Resolves alerts for several security advisories such as: GHSA-r6q2-hw4h-h46w GHSA-8qq5-rm4j-mr97

As per our assessment those vulnerabilities are not exploitable in the context of UI5 CLI.

The override for tar is specifically defined where necessary in order to not downgrade tar in case a new major version is being used by other dependencies.

@matz3 matz3 requested a review from a team January 23, 2026 08:38
Resolves alerts for several security advisories such as:
GHSA-r6q2-hw4h-h46w
GHSA-8qq5-rm4j-mr97

As per our assessment those vulnerabilities are not exploitable in the context of UI5 CLI.

The override for tar is specifically defined where necessary in order to not downgrade tar
in case a new major version is being used by other dependencies.
@matz3 matz3 force-pushed the override-tar-version branch from 16d490e to bd9ed4c Compare January 23, 2026 12:21
@coveralls
Copy link

Coverage Status

coverage: 96.262%. remained the same
when pulling bd9ed4c on override-tar-version
into 91ca031 on v4.

@matz3
Copy link
Member Author

matz3 commented Jan 23, 2026

I had to simplify the overrides so that it also works with npm 10, which apparently does not support defining an nested override for a package multiple times. This should still prevent from overriding the usage of a future v8 of tar by only applying the override to the packages where it is currently necessary.

Copy link
Contributor

@d3xter666 d3xter666 left a comment

Choose a reason for hiding this comment

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

LGTM

@matz3 matz3 merged commit c5e1221 into v4 Jan 23, 2026
21 checks passed
@matz3 matz3 deleted the override-tar-version branch January 23, 2026 12:35
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.

5 participants