Skip to content

fix: bound postgres query results#181

Merged
brendanjryan merged 3 commits intomainfrom
codex/query-result-bounds
May 4, 2026
Merged

fix: bound postgres query results#181
brendanjryan merged 3 commits intomainfrom
codex/query-result-bounds

Conversation

@brendanjryan
Copy link
Copy Markdown
Contributor

@brendanjryan brendanjryan commented May 3, 2026

Summary

Enforce Postgres query row bounds while streaming results, preserve safe LIMIT injection around trailing comments, and include a changelog entry.

@brendanjryan brendanjryan marked this pull request as ready for review May 3, 2026 20:53
@brendanjryan brendanjryan force-pushed the codex/query-result-bounds branch from e5f2bd7 to b35f1c3 Compare May 4, 2026 00:35
@brendanjryan brendanjryan merged commit 3405116 into main May 4, 2026
7 checks passed
@brendanjryan brendanjryan deleted the codex/query-result-bounds branch May 4, 2026 01:13
brendanjryan added a commit that referenced this pull request May 4, 2026
This PR was opened by the Changelogs release workflow.

When you're ready to release, merge this PR and the packages will be
published.

---

## `tidx@0.5.5`

### Patch Changes

- Harden PostgreSQL SQL validation by fixing CTE scope handling,
schema-qualified table checks, recursive depth accounting, LIMIT ALL
rejection, and traversal of previously unchecked AST clauses. (by
@brendanryan, [#179](#179))
- Validate public ClickHouse queries, block system catalogs and
dangerous table functions, enforce ClickHouse request timeouts, and
validate view SELECT SQL before execution. (by @brendanryan,
[#180](#180))
- Bound PostgreSQL query result processing by streaming rows with a hard
request limit and appending automatic LIMIT clauses on a separate line.
(by @brendanryan, [#181](#181))
- Hardened view administration by failing closed for trusted CIDR
checks, rejecting malformed CIDR configuration, hot-reloading active
trusted CIDRs, and requiring an explicit admin mutation header. (by
@brendanryan, [#182](#182))

Co-authored-by: brendanjryan <1572504+brendanjryan@users.noreply.github.com>
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