fix(raster): bound direct PostGIS import payloads - #3131
Conversation
![github-code-quality[bot]](https://avatars.githubusercontent.com/u/9919?s=60&v=4){kind=small}
|
@codex review |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: a5dcece713
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: f109806948
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@codex review |
|
Codex Review: Didn't find any major issues. Another round soon, please! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
|
@codex review |
|
Codex Review: Didn't find any major issues. Bravo. Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
mikemcdougall
added
train:landing
|
Merge train escalated this batch to a human: CI failure not attributable to a single member diff (and not autofixable). This PR is held out of future batches until the train:escalated label is removed. |
mikemcdougall
added
train:landing
1 participant
Pull Request
Issue Link
Related to #3098
Summary
Bounds the legacy synchronous
ST_FromGDALRasterimport path before it can materialize an oversized raster in the AOT serving process. This is the fail-closed safety slice; the durable staged worker pipeline remains open in RAST-014.Changes Made
File.ReadAllBytesAsyncwith a length-admitted exact reader that also rejects concurrent file growth.Testing
Gate Impact
Docs or Contract Impact
Release/Deploy Impact
Breaking Changes
None. Inputs above the legacy direct-import ceiling are now rejected fail-closed until the staged durable path lands.
Pre-PR Checklist
scripts/ci/pre-pr-check.shand all checks passedtype: description (#issue)OPENAPI_BREAKING_CHANGE_APPROVED— intentional Admin API break acknowledged for this PR