Skip to content

fix: detected calls to child_process from a function... in index.ts...#5327

Open
orbisai0security wants to merge 1 commit into
microsoft:mainfrom
orbisai0security:fix-javascript.lang.security.detect-child-process.detect-child-process-scripts-lib-index.ts
Open

fix: detected calls to child_process from a function... in index.ts...#5327
orbisai0security wants to merge 1 commit into
microsoft:mainfrom
orbisai0security:fix-javascript.lang.security.detect-child-process.detect-child-process-scripts-lib-index.ts

Conversation

@orbisai0security
Copy link
Copy Markdown

Summary

Fix high severity security issue in scripts/lib/index.ts.

Vulnerability

Field Value
ID javascript.lang.security.detect-child-process.detect-child-process
Severity HIGH
Scanner semgrep
Rule javascript.lang.security.detect-child-process.detect-child-process
File scripts/lib/index.ts:10

Description: Detected calls to child_process from a function argument command. This could lead to a command injection if the input is user controllable. Try to avoid calls to child_process, and if it is needed ensure user input is correctly sanitized or sandboxed.

Changes

  • scripts/ci/build-monaco-editor-core-pkg.ts
  • scripts/lib/index.ts

Verification

  • Build passes
  • Scanner re-scan confirms fix
  • LLM code review passed

Automated security fix by OrbisAI Security

Detected calls to child_process from a function argument `command`
Addresses javascript.lang.security.detect-child-process.detect-child-process
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant