Skip to content

Document the Query API relationship to service IP access lists - #955

Merged
sdairs merged 2 commits into
codex/focused-919from
codex/focused-922
Sep 17, 2026
Merged

sdairs merged 2 commits into
codex/focused-919from
codex/focused-922

Conversation

@sdairs

@sdairs sdairs commented Sep 15, 2026

Copy link
Copy Markdown
Collaborator

Summary

Explain the Query API's proxied route and why the service IP access list for direct connections does not filter that route. The documentation also names the controls that still apply: API-key IP lists, endpoint binding, database roles and OAuth read-only authorization.

Help remains within its context budget; output-format details are retained in README. Runtime requests and authentication are unchanged.

Fixes #922.

Validation

Verified against official Query API IP access documentation. Whole-command-tree and query-context structural tests, formatting and diff checks passed.

Stack

Appended to native stack 876.

Combined local validation

Combined stack head e20d63b3af4c8541a6f597d9494b530f2c56b666 passed all required local checks: formatting, CLI clippy/tests, telemetry-disabled check and all-target clippy, API/analyzer all-target clippy/tests, all-feature workspace build, Python classifier/drift tests, and diff checks.

Test totals: 1991 CLI, 672 API/analyzer, and 93 Python tests (2756 total). Ignored/live tests are excluded from these totals.

Snapshot API access was revalidated successfully on the exact heads of PRs #940 and #944 after the environment feature flag was enabled. Run links and coverage details are recorded in tracker #939.

Stack order, exact heads and explicit issue Development relationships were verified in the implementation tracker.

@sdairs
sdairs added this pull request to stack #876 September 15, 2026 15:35
@sdairs sdairs linked an issue Sep 15, 2026 that may be closed by this pull request
@sdairs
sdairs marked this pull request as ready for review September 16, 2026 13:14
Carry existing lower-stack README corrections and ancestry forward. The merged tree matches the independently verified cumulative stack result.

@iskakaushik iskakaushik left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sdairs
sdairs merged commit a65a9ca into main Sep 17, 2026
17 checks passed
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.

Document the Query API relationship to service IP access lists

2 participants