chore: use replaceAll instead of global regex to redact simple string (#6396)

Author: wraithgar

scripts/create-node-pr.js

@@ -280,5 +280,5 @@ const main = async (spec, branch = 'main', opts) => withTempDir(CWD, async (tmpD
 })
 
 run(({ argv, ...opts }) => main(argv.remain[0], argv.remain[1], opts), {
- redact: new RegExp(process.env.GITHUB_TOKEN, 'g'),
+ redact: process.env.GITHUB_TOKEN,
 })

scripts/util.js

@@ -149,7 +149,7 @@ const run = async (main, { redact } = {}) => {
  process.on('log', (l, ...args) => {
  if (argv.debug || process.env.CI || defaultLevels.includes(l)) {
  for (const line of formatWithOptions({ colors: true }, ...args).split('\n')) {
- const redacted = redact ? line.replace(redact, '***') : line
+ const redacted = redact ? line.replaceAll(redact, '***') : line
  // eslint-disable-next-line no-console
  console.error(l.slice(0, 4).toUpperCase(), redacted)
  }