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

Update npm registry keys for Corepack #1010

Merged
merged 3 commits into from
Feb 3, 2025
Merged

Conversation

colincasey
Copy link
Contributor

This is a workaround for Node versions that bundle a version of Corepack that is affected by recent changes to npm's public signing keys.

This is a workaround for Node versions that bundle a version of Corepack that is affected by recent changes to npm's public signing keys.
@colincasey colincasey self-assigned this Feb 3, 2025
@colincasey colincasey requested a review from a team as a code owner February 3, 2025 16:13
@colincasey colincasey enabled auto-merge (squash) February 3, 2025 16:48
@colincasey colincasey merged commit 7e62cda into main Feb 3, 2025
41 checks passed
@colincasey colincasey deleted the update_corepack_keys branch February 3, 2025 17:27
colincasey added a commit that referenced this pull request Feb 3, 2025
* main:
  Update npm registry keys for Corepack (#1010)
heroku-linguist bot added a commit that referenced this pull request Feb 3, 2025
## heroku/nodejs

### Changed

- Updated `heroku/nodejs-corepack` to `3.4.5`.
- Updated `heroku/nodejs-engine` to `3.4.5`.
- Updated `heroku/nodejs-npm-engine` to `3.4.5`.
- Updated `heroku/nodejs-npm-install` to `3.4.5`.
- Updated `heroku/nodejs-pnpm-engine` to `3.4.5`.
- Updated `heroku/nodejs-pnpm-install` to `3.4.5`.
- Updated `heroku/nodejs-yarn` to `3.4.5`.

## heroku/nodejs-corepack

### Changed

- Updated npm keys for Corepack versions `0.27`, `0.28`, `0.29`, `0.30`. ([#1010](#1010))

## heroku/nodejs-engine

### Added

- 23.7.0 (linux-amd64, linux-arm64)

## heroku/nodejs-function

### Changed

- Updated `heroku/nodejs-engine` to `3.4.5`.
- Updated `heroku/nodejs-function-invoker` to `3.4.5`.
- Updated `heroku/nodejs-npm` to `3.4.5`.

## heroku/nodejs-function-invoker

- No changes.

## heroku/nodejs-npm

- No changes.

## heroku/nodejs-npm-engine

- No changes.

## heroku/nodejs-npm-install

- No changes.

## heroku/nodejs-pnpm-engine

- No changes.

## heroku/nodejs-pnpm-install

- No changes.

## heroku/nodejs-yarn

- No changes.
@heroku-linguist heroku-linguist bot mentioned this pull request Feb 3, 2025
heroku-linguist bot added a commit that referenced this pull request Feb 4, 2025
## heroku/nodejs

### Changed

- Updated `heroku/nodejs-corepack` to `3.4.5`.
- Updated `heroku/nodejs-engine` to `3.4.5`.
- Updated `heroku/nodejs-npm-engine` to `3.4.5`.
- Updated `heroku/nodejs-npm-install` to `3.4.5`.
- Updated `heroku/nodejs-pnpm-engine` to `3.4.5`.
- Updated `heroku/nodejs-pnpm-install` to `3.4.5`.
- Updated `heroku/nodejs-yarn` to `3.4.5`.

## heroku/nodejs-corepack

### Changed

- Updated npm keys for Corepack versions `0.27`, `0.28`, `0.29`, `0.30`. ([#1010](#1010))

## heroku/nodejs-engine

### Added

- 23.7.0 (linux-amd64, linux-arm64)

## heroku/nodejs-function

### Changed

- Updated `heroku/nodejs-engine` to `3.4.5`.
- Updated `heroku/nodejs-function-invoker` to `3.4.5`.
- Updated `heroku/nodejs-npm` to `3.4.5`.

## heroku/nodejs-function-invoker

- No changes.

## heroku/nodejs-npm

- No changes.

## heroku/nodejs-npm-engine

- No changes.

## heroku/nodejs-npm-install

- No changes.

## heroku/nodejs-pnpm-engine

- No changes.

## heroku/nodejs-pnpm-install

- No changes.

## heroku/nodejs-yarn

- No changes.

Co-authored-by: heroku-linguist[bot] <136119646+heroku-linguist[bot]@users.noreply.github.com>
heroku-linguist bot added a commit to heroku/cnb-builder-images that referenced this pull request Feb 4, 2025
## heroku/nodejs

### Changed

- Updated `heroku/nodejs-corepack` to `3.4.5`.
- Updated `heroku/nodejs-engine` to `3.4.5`.
- Updated `heroku/nodejs-npm-engine` to `3.4.5`.
- Updated `heroku/nodejs-npm-install` to `3.4.5`.
- Updated `heroku/nodejs-pnpm-engine` to `3.4.5`.
- Updated `heroku/nodejs-pnpm-install` to `3.4.5`.
- Updated `heroku/nodejs-yarn` to `3.4.5`.

## heroku/nodejs-corepack

### Changed

- Updated npm keys for Corepack versions `0.27`, `0.28`, `0.29`, `0.30`. ([#1010](heroku/buildpacks-nodejs#1010))

## heroku/nodejs-engine

### Added

- 23.7.0 (linux-amd64, linux-arm64)

## heroku/nodejs-function

### Changed

- Updated `heroku/nodejs-engine` to `3.4.5`.
- Updated `heroku/nodejs-function-invoker` to `3.4.5`.
- Updated `heroku/nodejs-npm` to `3.4.5`.

## heroku/nodejs-function-invoker

- No changes.

## heroku/nodejs-npm

- No changes.

## heroku/nodejs-npm-engine

- No changes.

## heroku/nodejs-npm-install

- No changes.

## heroku/nodejs-pnpm-engine

- No changes.

## heroku/nodejs-pnpm-install

- No changes.

## heroku/nodejs-yarn

- No changes.
heroku-linguist bot added a commit to heroku/cnb-builder-images that referenced this pull request Feb 4, 2025
## heroku/nodejs

### Changed

- Updated `heroku/nodejs-corepack` to `3.4.5`.
- Updated `heroku/nodejs-engine` to `3.4.5`.
- Updated `heroku/nodejs-npm-engine` to `3.4.5`.
- Updated `heroku/nodejs-npm-install` to `3.4.5`.
- Updated `heroku/nodejs-pnpm-engine` to `3.4.5`.
- Updated `heroku/nodejs-pnpm-install` to `3.4.5`.
- Updated `heroku/nodejs-yarn` to `3.4.5`.

## heroku/nodejs-corepack

### Changed

- Updated npm keys for Corepack versions `0.27`, `0.28`, `0.29`, `0.30`. ([#1010](heroku/buildpacks-nodejs#1010))

## heroku/nodejs-engine

### Added

- 23.7.0 (linux-amd64, linux-arm64)

## heroku/nodejs-function

### Changed

- Updated `heroku/nodejs-engine` to `3.4.5`.
- Updated `heroku/nodejs-function-invoker` to `3.4.5`.
- Updated `heroku/nodejs-npm` to `3.4.5`.

## heroku/nodejs-function-invoker

- No changes.

## heroku/nodejs-npm

- No changes.

## heroku/nodejs-npm-engine

- No changes.

## heroku/nodejs-npm-install

- No changes.

## heroku/nodejs-pnpm-engine

- No changes.

## heroku/nodejs-pnpm-install

- No changes.

## heroku/nodejs-yarn

- No changes.

Co-authored-by: heroku-linguist[bot] <136119646+heroku-linguist[bot]@users.noreply.github.com>
pranavrajs pushed a commit to chatwoot/chatwoot that referenced this pull request Feb 5, 2025
So, a while back Circle CI builds and Heroku builds started to fail. 

From all the threads I read, it seems like the [npm registry rotated
it's signing
keys](pnpm/pnpm#9014 (comment))
New pnpm versions were signed with the new key. Corepack, however,
bundles a static set of trusted keys (from Node’s release), so it
continued verifying signatures only against the old key. When it
encountered packages signed with the new key, Corepack’s integrity check
failed with “Cannot find matching keyid” errors.This mismatch caused
Corepack’s integrity check to fail with “Cannot find matching keyid”
errors.

Workarounds include the following

1. Updating Corepack (to 0.31.0), they [upgraded their
package](https://github.com/nodejs/corepack/releases/tag/v0.31.0) to
include the new integrity check keys. But we seldom control what's going
on with the CI, also, updating this across our scripts is going to be a
painful task. Besides Heroku has [made some
fixes](heroku/buildpacks-nodejs#1010) around
this
2. Disabling integrity checks 🔥 #YOLO
3. Pinning `pnpm` to older versions, or pinning it to a newer version
with the checksum in place.

Doing the third one here, running `corepack use [email protected]` fixes this,
[ref](pnpm/pnpm#9014 (comment))

We can get rid of this over time as CDN caches used by build systems are
refreshed. But the change in this PR is not disruptive in anyway, only
rigidly secure.

Fixes: #10832

---

Here are the threads to follow 

- pnpm/pnpm#9014
- pnpm/pnpm#9029
- nodejs/corepack#612
- nodejs/corepack#616
- heroku/buildpacks-nodejs#1010

---------

Co-authored-by: Vishnu Narayanan <[email protected]>
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