Address potential classloader performance issues in JS scripts

[kingthorin]

No description provided.

[kingthorin]

Note I used const in all the changes but didn't change/reduce other use of var. I can I just wasn't sure if it should be the same PR.

[psiinon]

Checkmarx One – Scan Summary & Details – fc6b0dca-a0ad-49eb-906d-c19efb0bba81

New Issues (5)

Checkmarx found the following issues in this Pull Request

Severity	Issue	Source File / Package	Checkmarx Insight
	Last User Is 'root'	/docker-wrapper: 10	details Leaving the last user as root can cause security risks. Change to another user after running the commands that need privileges ID: 48tNdC6UziXyOGUccQZn3tPPzi4%3D
	MAINTAINER Instruction Being Used	/docker-wrapper: 3	details The MAINTAINER instruction sets the Author field of the generated images. The LABEL instruction is a much more flexible version of this and you sh... ID: nlHBIHIr9RZHoVXOgGxJ9hQCHFA%3D
	Unpinned Actions Full Length Commit SHA	/codeql.yml: 31	details Pinning an action to a full length commit SHA is currently the only way to use an action as an immutable release. Pinning to a particular SHA help... ID: z89ONTXYaYdPcNUEzfFqPVDqGfU%3D
	Unpinned Actions Full Length Commit SHA	/codeql.yml: 34	details Pinning an action to a full length commit SHA is currently the only way to use an action as an immutable release. Pinning to a particular SHA help... ID: wmF9HbZcEd4Px83a0Vg%2BO%2F%2B%2B4BU%3D
	Unpinned Actions Full Length Commit SHA	/codeql.yml: 35	details Pinning an action to a full length commit SHA is currently the only way to use an action as an immutable release. Pinning to a particular SHA help... ID: ivv4LqDvobLaIQBf4po7RJO0z9E%3D

---

Use @Checkmarx to reach out to us for assistance.

Just send a PR comment with @Checkmarx followed by a natural language request.

Examples: @Checkmarx how are you able to help me? @Checkmarx rescan this PR

[kingthorin]

The CX failure is unrelated to the changes.