Skip to content

[stable33] fix: Add explicit dependency for vue-loader@legacy - #1497

Merged
danxuliu merged 1 commit into
stable33from
backport/1496/stable33
Aug 7, 2026
Merged

[stable33] fix: Add explicit dependency for vue-loader@legacy#1497
danxuliu merged 1 commit into
stable33from
backport/1496/stable33

Conversation

@backportbot

@backportbot backportbot Bot commented Aug 7, 2026

Copy link
Copy Markdown

Backport of PR #1496

@backportbot
backportbot Bot requested review from ShGKme and danxuliu August 7, 2026 04:16
@backportbot backportbot Bot added this to the Nextcloud 33.0.8 milestone Aug 7, 2026
Current vue-loader@legacy installed as transitive has a vulnerable
dependency, and there is no fix for the installed major version. When
"npm audit fix" is run it tries to update vue-loader to another major,
because no version is explicitly specified in the project. Fortunately,
the postinstall script of webpack-vue-config verifies that the right
version of vue-loader is installed and aborts the process (with a nice
warning) if it is not.

However, due to all of the above, when "npm audit fix" is run by the
workflow the command fails and no fix is applied. To solve that now an
explicit dependency is added again for vue-loader@legacy.

Note that the explicit dependency was removed in the past due to being
automatically handled when the dependencies are installed, but it is
necessary when nothing is installed and the dependency has a
vulnerability.

Thanks ShGKme for the explanation!

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
@danxuliu
danxuliu force-pushed the backport/1496/stable33 branch from fb19703 to 9482d71 Compare August 7, 2026 04:29
@danxuliu
danxuliu merged commit 9154025 into stable33 Aug 7, 2026
34 checks passed
@danxuliu
danxuliu deleted the backport/1496/stable33 branch August 7, 2026 04:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant