Skip to content

fix: block remote CSV redirects - #3276

Merged
im-anishraj merged 0 commit into
im-anishraj:mainfrom
LuisFigueroaG:fix-remote-csv-disable-redirects
Jul 22, 2026
Merged

fix: block remote CSV redirects#3276
im-anishraj merged 0 commit into
im-anishraj:mainfrom
LuisFigueroaG:fix-remote-csv-disable-redirects

Conversation

@LuisFigueroaG

Copy link
Copy Markdown
Contributor

Summary

  • fetch remote CSV URLs with an opener that rejects HTTP redirects instead of following them implicitly
  • surface redirect responses as RemoteReadError with the original URL and HTTP status
  • add a local HTTP 302 regression test for remote CSV fetches

Closes #2614

Validation

  • uv run pytest -q tests/test_remote_csv.py
  • uv run ruff format --check arnio/io.py tests/test_remote_csv.py
  • uv run ruff check arnio/io.py tests/test_remote_csv.py
  • uv run ruff check .
  • uv run pytest -q
  • git diff --check

Known existing environment/project issues:

  • uv run ruff format --check . reports pre-existing formatting changes in 33 unrelated files
  • uv run pyright fails because the pyright executable is not installed in the uv environment

@vercel

vercel Bot commented Jul 8, 2026

Copy link
Copy Markdown

@LuisFigueroaG is attempting to deploy a commit to the xtylishanish-gmailcom's projects Team on Vercel.

A member of the Team first needs to authorize it.

@im-anishraj im-anishraj added gssoc:approved Merged and approved GSSoC contribution quality:clean GSSoC scoring label for clean, well-scoped merged PRs labels Jul 20, 2026
@im-anishraj
im-anishraj merged commit 06dff89 into im-anishraj:main Jul 22, 2026
49 of 50 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gssoc:approved Merged and approved GSSoC contribution quality:clean GSSoC scoring label for clean, well-scoped merged PRs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CRITICAL: Server-Side Request Forgery (SSRF) and Arbitrary File Read in \ ead_csv\ Remote URL Fetching

2 participants