Skip to content

ai-gov: add query params to org policy - #26551

Open
craig-osterhout wants to merge 2 commits into
docker:mainfrom
craig-osterhout:gov-org-query-params
Open

craig-osterhout wants to merge 2 commits into
docker:mainfrom
craig-osterhout:gov-org-query-params

Conversation

@craig-osterhout

@craig-osterhout craig-osterhout commented Nov 5, 2026 •

Copy link
Copy Markdown
Contributor

Description

Query params can now be specified in org policy.

Noticed the existing http content could use some examples and a restructure. Didn't want to scope creep into this change, so will follow up and refresh the topic once this goes live.

https://deploy-preview-26551--docsdocker.netlify.app/ai/governance/organization/policies/network/#http-rules

https://deploy-preview-26551--docsdocker.netlify.app/ai/governance/reference/network-rules/#query-parameters

Related issues or tickets

Reviews

  • Technical review
  • Editorial review
  • Product review

Signed-off-by: Craig Osterhout <craig.osterhout@docker.com>
@netlify

netlify Bot commented Nov 5, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit 7fdf64f
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/6aecc9debbe2bebb29ad2ddd
😎 Deploy Preview https://deploy-preview-26551--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@derekmisler derekmisler left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

left 1 nitpick inline. checked the copy against the org policy composer in docker/frontends and it matches, including the Enter and & entry handling and the query set applying to every path pattern.

other things i noticed: the composer rejects query parameters on CIDR destinations and requires host or host:port (bracketed IPv6) when query parameters are set. neither is in the docs, so you could mention them here or in a follow-up.

Co-authored-by: Derek Misler <derekmisler@gmail.com>
@craig-osterhout
craig-osterhout requested a review from a team November 5, 2026 18:24
@craig-osterhout craig-osterhout added the status/review Pull requests that are ready for review label Nov 5, 2026

@bschuck-docker bschuck-docker left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/ai status/review Pull requests that are ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants