[APS-19415][APS-19417] fix: bump browserstack-local + override critical transitive CVEs#34
Open
Rohannagariya1 wants to merge 1 commit into
Conversation
…S-19415][APS-19417] APS-19415 (browserstack-local command injection, GHSA-g4w6-c99w-4wh7): - Narrow browserstack-local ^1.0.0 -> ^1.5.11; lock resolves 1.5.13. Clears the <=1.5.8 CVE and removes the dangerously broad 1.x range. APS-19417 (EOL protractor pulls critical transitive CVEs) -- INTERIM: - Add npm overrides forcing json-schema >=0.4.0 (GHSA-896r-f27r-55mw) and minimist >=1.2.6 (GHSA-xvch-5gv4-984h). Lock resolves json-schema 0.4.0 and minimist 1.2.8. - This clears 3 critical findings (json-schema, jsprim, minimist). - protractor itself remains EOL; full Playwright/WebdriverIO migration is a separate sprint (APPSEC-409), OUT OF SCOPE here. npm audit: critical 5 -> 2, total 27 -> 23. The 2 remaining criticals (form-data, request) are protractor's own deep transitive chain and cannot be cleared without removing protractor. Resolves: APS-19415, APS-19417 Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Security Fix: APS-19415 + APS-19417 (combined)
This single PR addresses two linked findings on this repo. Full protractor -> Playwright/WebdriverIO migration is out of scope (tracked in APPSEC-409); this PR is the interim CVE-clearing fix.
APS-19415 — browserstack-local command injection (GHSA-g4w6-c99w-4wh7, Medium)
package.jsondeclaredbrowserstack-local: ^1.0.0, resolving to<=1.5.8(vulnerable; command injection via unsanitizedlogfileoption).^1.5.11. Lock now resolves 1.5.13 (latest 1.x). Clears the CVE and removes the dangerously broad1.xrange.APS-19417 — EOL protractor pulls critical transitive CVEs (High) — INTERIM
^5.4.4is EOL (Aug 2023) and drags injson-schema <0.4.0(GHSA-896r-f27r-55mw, critical) andminimist 1.0.0–1.2.5(GHSA-xvch-5gv4-984h, critical).overridesforcingjson-schema >=0.4.0andminimist >=1.2.6. Lock now resolves json-schema 0.4.0 and minimist 1.2.8."dependencies": { - "browserstack-local": "^1.0.0", + "browserstack-local": "^1.5.11", "protractor": "^5.4.4" }, + "overrides": { + "json-schema": ">=0.4.0", + "minimist": ">=1.2.6" + },Validation —
npm auditbefore vs after (lock refreshed vianpm install --package-lock-only)<=1.5.8)<0.4.0)1.0.0–1.2.5)Critical set went from
[form-data, json-schema, jsprim, minimist, request]->[form-data, request]. The 3 targeted findings (browserstack-local, json-schema, minimist) plus jsprim are fully cleared.Remaining
form-data+requestcriticals are protractor's own deep transitive chain (protractor -> webdriver-manager -> request -> form-data) and cannot be cleared without removing protractor — that requires the APPSEC-409 migration and is out of scope for this interim fix.Testing
npm install --package-lock-onlyrefresh: PASS (exit 0); lock verified to resolve the bumped/overridden versions.npm auditbefore/after: captured above; targeted CVEs cleared.Jira Tickets
Checklist