Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: always ignore ownership from tar headers #261

Merged
merged 1 commit into from Feb 21, 2023
Merged

Conversation

nlf
Copy link
Contributor

@nlf nlf commented Feb 16, 2023

closes npm/cli#6173

@nlf nlf requested a review from a team as a code owner February 16, 2023 20:43
@nlf nlf requested review from lukekarrys and removed request for a team February 16, 2023 20:43
@melroy89
Copy link

Yes, please!

@EvanCarroll
Copy link

If anyone else is having this issue. Just be aware of this release of pacote not yet being in npm despite it being released.

  • npm bundles it's pacote dependency in node_modules/
  • That bundle in npm 9.5.0 doesn't include this fix.

You can force the update with npm install -g pacote@latest

@rbalet
Copy link

rbalet commented Mar 16, 2023

You can force the update with npm install -g pacote@latest

Didn't fixed the problem for me

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.

[BUG] npm installs package with wrong user/group ownership
5 participants