retry npm install/view for appsec and plugins tests (#5434)

rochdev · web-flow · commit f6eefc599350 · 2025-03-20T22:16:27.000-04:00
diff --git a/.github/workflows/appsec.yml b/.github/workflows/appsec.yml
@@ -257,7 +257,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
-      - run: yarn install
+      - uses: ./.github/actions/install
       - uses: ./.github/actions/node/oldest-maintenance-lts
       - run: yarn test:integration:appsec
       - uses: ./.github/actions/node/active-lts
diff --git a/scripts/install_plugin_modules.js b/scripts/install_plugin_modules.js
@@ -161,12 +161,11 @@ async function getVersionList (name) {
   return list
 }
 
-function npmView (input) {
+function npmView (input, retry = true) {
   return new Promise((resolve, reject) => {
     childProcess.exec(`npm view ${input} --json`, (err, stdout) => {
       if (err) {
-        reject(err)
-        return
+        return retry ? npmView(input, false).then(resolve, reject) : reject(err)
       }
       resolve(JSON.parse(stdout.toString('utf8')))
     })

Original file line number	Diff line number	Diff line change
`@@ -161,12 +161,11 @@ async function getVersionList (name) {`
`161`	`161`	`return list`
`162`	`162`	`}`
`163`	`163`
`164`		`-function npmView (input) {`
	`164`	`+function npmView (input, retry = true) {`
`165`	`165`	`return new Promise((resolve, reject) => {`
`166`	`166`	childProcess.exec(`npm view ${input} --json`, (err, stdout) => {
`167`	`167`	`if (err) {`
`168`		`- reject(err)`
`169`		`- return`
	`168`	`+ return retry ? npmView(input, false).then(resolve, reject) : reject(err)`
`170`	`169`	`}`
`171`	`170`	`resolve(JSON.parse(stdout.toString('utf8')))`
`172`	`171`	`})`