←back to thread

441 points longcat | 3 comments | | HN request time: 0.532s | source
1. BobbyTables2 ◴[] No.45039788[source]
ELI5, how was the malicious PR approved and merged?

Are they using AI for automated code review too?

replies(2): >>45039871 #>>45046546 #
2. danr4 ◴[] No.45039871[source]
seems like the npm repo got hacked and the compromised version was just uploaded
3. david_allison ◴[] No.45046546[source]
The workflows were set up to execute with a read/write `GITHUB_TOKEN` for `nx` when a PR was created/edited (no approval necessary).

See the security warnings on `pull_request_target`

https://docs.github.com/en/actions/reference/workflows-and-a...

https://securitylab.github.com/resources/github-actions-prev...