Skip to content

DO NOT MERGE: Test adding vulnerability to pyproject but not uv.lock#226

Open
sumau wants to merge 1 commit into
mainfrom
test-dependency-review
Open

DO NOT MERGE: Test adding vulnerability to pyproject but not uv.lock#226
sumau wants to merge 1 commit into
mainfrom
test-dependency-review

Conversation

@sumau

@sumau sumau commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Test adding vulnerability to pyproject.yaml but not to uv.lock file. Dependency review flags PR but not Datadog.

@sumau
sumau requested a review from a team as a code owner July 16, 2026 10:32
@github-actions

Copy link
Copy Markdown

Dependency Review

The following issues were found:
  • ❌ 1 vulnerable package(s)
See the Details below.

Vulnerabilities

pyproject.toml

NameVersionVulnerabilitySeverity
aiohttp3.7.0AIOHTTP's HTTP Parser auto_decompress feature is vulnerable to zip bombhigh
aiohttp vulnerable to Denial of Service when trying to parse malformed POST requestshigh
aiohttp is vulnerable to directory traversalhigh
AIOHTTP vulnerable to DoS when bypassing assertsmoderate
AIOHTTP vulnerable to denial of service through large payloadsmoderate
AIOHTTP vulnerable to DoS through chunked messagesmoderate
aiohttp allows unlimited trailer headers, leading to possible uncapped memory usagemoderate
AIOHTTP affected by UNC SSRF/NTLMv2 Credential Theft/Local File Read in static resource handler on Windowsmoderate
AIOHTTP has a Multipart Header Size Bypassmoderate
AIOHTTP accepts duplicate Host headersmoderate
AIOHTTP is Vulnerable to Deserialization of Untrusted Datamoderate
AIOHTTP is vulnerable to cross-origin redirect with per-request cookiesmoderate
aiohttp: DigestAuthMiddleware Applies Credentials to Cross-Origin Redirect Challengesmoderate
aiohttp: C HTTP Parser Bypasses max_line_size for Fragmented Linesmoderate
aiohttp: Unread Compressed Request Bodies Bypass client_max_size During Cleanupmoderate
aiohttp: HTTP/1 Pipelined Requests Queue Without Limitmoderate
aiohttp: Incomplete websocket frame payloads bypass memory limitsmoderate
aiohttp.web.Application vulnerable to HTTP request smuggling via llhttp HTTP request parsermoderate
AIOHTTP has problems in HTTP parser (the python one, not llhttp)moderate
aiohttp has vulnerable dependency that is vulnerable to request smugglingmoderate
aiohttp allows request smuggling due to incorrect parsing of chunk extensionsmoderate
aiohttp Cross-site Scripting vulnerability on index pages for static file handlingmoderate
aiohttp's ClientSession is vulnerable to CRLF injection via methodmoderate
aiohttp's ClientSession is vulnerable to CRLF injection via versionmoderate
aiohttp's HTTP parser (the python one, not llhttp) still overly lenient about separatorsmoderate
`aiohttp` Open Redirect vulnerability (`normalize_path_middleware` middleware)low
aiohttp: TLS Server Hostname Override Is Ignored When Reusing HTTPS Connectionslow
aiohttp: Payload Response Resources Are Not Closed After Mid-Body Disconnectlow
Aiohttp has inconsistent interpretation of `Content-Length` vs. `Transfer-Encoding` differing in C and Python fallbackslow
aiohttp: Host-Only Cookies Become Domain Cookies After CookieJar Persistencelow
aiohttp: CRLF injection in multipart headerslow
AIOHTTP's C parser (llhttp) accepts null bytes and control characters in response header values - header injection/security bypasslow
AIOHTTP has HTTP response splitting via \r in reason phraselow
AIOHTTP leaks Cookie and Proxy-Authorization headers on cross-origin redirectlow
AIOHTTP has late size enforcement for non-file multipart fields causes memory DoSlow
AIOHTTP has CRLF injection through multipart part content type header constructionlow
AIOHTTP Affected by Denial of Service (DoS) via Unbounded DNS Cache in TCPConnectorlow
AIOHTTP is vulnerable to HTTP Request/Response Smuggling through incorrect parsing of chunked trailer sectionslow
AIOHTTP Vulnerable to Cookie Parser Warning Stormlow
AIOHTTP's unicode processing of header values could cause parsing discrepancieslow
AIOHTTP has unicode match groups in regexes for ASCII protocol elementslow
AIOHTTP vulnerable to brute-force leak of internal static file path componentslow

OpenSSF Scorecard

PackageVersionScoreDetails
pip/aiohttp 3.7.0 🟢 7.1
Details
CheckScoreReason
Maintained🟢 1030 commit(s) and 13 issue activity found in the last 90 days -- score normalized to 10
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Code-Review🟢 4Found 7/15 approved changesets -- score normalized to 4
CII-Best-Practices⚠️ 0no effort to earn an OpenSSF best practices badge detected
Token-Permissions⚠️ 0detected GitHub workflow tokens with excessive permissions
Binary-Artifacts🟢 10no binaries found in the repo
License🟢 10license file detected
Fuzzing🟢 10project is fuzzed
Branch-Protection⚠️ -1internal error: error during branchesHandler.setup: internal error: some github tokens can't read classic branch protection rules: https://github.com/ossf/scorecard-action/blob/main/docs/authentication/fine-grained-auth-token.md
Security-Policy🟢 10security policy file detected
Pinned-Dependencies⚠️ 0dependency not pinned by hash detected -- score normalized to 0
Packaging🟢 10packaging workflow detected
SAST🟢 10SAST tool is run on all commits
Signed-Releases🟢 64 out of the last 5 releases have a total of 4 signed artifacts.

Scanned Files

  • pyproject.toml

@github-actions

Copy link
Copy Markdown

Your PR has commits that are missing the Signed-off-by trailer. This is likely due to the pre-commit hook not being configured on your local machine. The usual fix for this issue is to run pre-commit install --install-hooks --overwrite -t commit-msg -t pre-commit, however for more detailed help in setting up the pre-commit hooks, follow the instructions at https://github.com/uktrade/github-standards/blob/main/README.md#usage

@sumau sumau changed the title Test Dependency Review - DO NOT MERGE ME! DO NOT MERGE - Test Dependency Review by adding vulnerability to pyproject Jul 16, 2026
@sumau sumau changed the title DO NOT MERGE - Test Dependency Review by adding vulnerability to pyproject DO NOT MERGE: Test Dependency Review by adding vulnerability to pyproject Jul 16, 2026
@sumau sumau changed the title DO NOT MERGE: Test Dependency Review by adding vulnerability to pyproject DO NOT MERGE: Test by adding vulnerability to pyproject Jul 20, 2026
@sumau sumau changed the title DO NOT MERGE: Test by adding vulnerability to pyproject DO NOT MERGE: Test adding vulnerability to pyproject Jul 20, 2026
@sumau sumau changed the title DO NOT MERGE: Test adding vulnerability to pyproject DO NOT MERGE: Test adding vulnerability to pyproject but not uv.lock Jul 20, 2026
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