[Aikido] Fix security issue in fast-uri via minor version upgrade from 3.1.0 to 3.1.6 in apps - #12
Open
aikido-autofix[bot] wants to merge 1 commit into
Conversation
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 4e61a0e. Configure here.
|
|
||
| "when-exit": ["when-exit@2.1.5", "", {}, "sha512-VGkKJ564kzt6Ms1dbgPP/yuIoQCrsFAnRbptpC5wOEsDaNsbCB2bnfnaA8i/vRs5tjUSEOtIuvl9/MyVsvQZCg=="], | ||
|
|
||
| "ajv/fast-uri": ["fast-uri@3.1.0", "", {}, "sha512-iPeeDKJSWf4IEOasVVrknXpaBV0IApz/gp7S2bb7Z4Lljbl2MGJRqInZiUrQwV16cpzw/D3S5j5Julj/gT52AA=="], |
There was a problem hiding this comment.
Vulnerable fast-uri retained under ajv
High Severity
The lockfile adds fast-uri@3.1.6 as a workspace dependency while pinning ajv/fast-uri to 3.1.0. ajv is the only runtime consumer of fast-uri via conf, so the vulnerable parser remains on the nested install path and the advertised CVE fixes do not take effect.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit 4e61a0e. Configure here.
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.


Upgrade fast-uri to fix SSRF/host-bypass, path-traversal, authority-spoofing, and Unicode-normalization vulnerabilities in URI parsing and comparison.
✅ 11 CVEs resolved by this upgrade
This PR will resolve the following CVEs:
fast-uritreats as paths but Node's URL parser treats as authority, enabling redirection to unintended destinations.Note
Low Risk
Lockfile-only dependency patch with no CLI logic changes; residual risk is limited to whether nested
ajv/fast-uri@3.1.0remains reachable at runtime versus the pinned 3.1.6.Overview
Updates
apps/cli/bun.lockso@sharehtml/cliresolvesfast-uri@3.1.6instead of 3.1.0, adding it as a direct workspace dependency to enforce the patched version.This is a dependency-only security bump (Aikido/CVE remediation) addressing URI parsing and normalization issues—host allowlist/SSRF bypass, path traversal via encoded segments, authority spoofing, and IDN/IPv6 handling—in code paths that use
fast-uri(including viaconf→ajv).No application source changes in this diff. Note the lockfile still records
ajv/fast-uri@3.1.0as a nested copy underajv; the new direct pin is what pulls 3.1.6 for the CLI workspace.Reviewed by Cursor Bugbot for commit 4e61a0e. Bugbot is set up for automated code reviews on this repo. Configure here.