esm: fix loader hooks accepting too many arguments by JakobJingleheimer · Pull Request #44109 · nodejs/node

@JakobJingleheimer

@JakobJingleheimer added the loaders

Issues and PRs related to ES module loaders

label

Aug 2, 2022

cspotcode

GeoffreyBooth

aduh95

guybedford

@JakobJingleheimer

account for prototype pollution in Array & Object

JakobJingleheimer

aduh95

consolidate assignment & set arg0 default
validateArgs array → separate args
…arguments

ReflectApply → direct invocation w separate args

aduh95

aduh95

@JakobJingleheimer

@aduh95 aduh95 added author ready

PRs that have at least one approval, no pending requests for changes, and a CI started.

request-ci

Add this label to start a Jenkins CI on a PR.

commit-queue-squash

Add this label to instruct the Commit Queue to squash all the PR commits into the first one.

labels

Aug 3, 2022

aduh95

@aduh95 aduh95 removed the author ready

PRs that have at least one approval, no pending requests for changes, and a CI started.

label

Aug 3, 2022

@JakobJingleheimer

GeoffreyBooth

@MoLow MoLow added commit-queue

Add this label to land a pull request using GitHub Actions.

and removed commit-queue-failed

An error occurred while landing this pull request using GitHub Actions.

labels

Aug 4, 2022

targos pushed a commit that referenced this pull request

Aug 6, 2022
PR-URL: #44109
Reviewed-By: Geoffrey Booth <webadmin@geoffreybooth.com>
Reviewed-By: Guy Bedford <guybedford@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>

This was referenced

Aug 7, 2022

danielleadams pushed a commit that referenced this pull request

Aug 16, 2022
PR-URL: #44109
Reviewed-By: Geoffrey Booth <webadmin@geoffreybooth.com>
Reviewed-By: Guy Bedford <guybedford@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>

ruyadorno pushed a commit that referenced this pull request

Aug 23, 2022
PR-URL: #44109
Reviewed-By: Geoffrey Booth <webadmin@geoffreybooth.com>
Reviewed-By: Guy Bedford <guybedford@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>

Fyko pushed a commit to Fyko/node that referenced this pull request

Sep 15, 2022
PR-URL: nodejs#44109
Reviewed-By: Geoffrey Booth <webadmin@geoffreybooth.com>
Reviewed-By: Guy Bedford <guybedford@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>

guangwong pushed a commit to noslate-project/node that referenced this pull request

Oct 10, 2022
PR-URL: nodejs/node#44109
Reviewed-By: Geoffrey Booth <webadmin@geoffreybooth.com>
Reviewed-By: Guy Bedford <guybedford@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>