Skip to content

fix(dev): default adk.web.cors.origins to same-origin only instead of * - #1668

Open
copybara-service[bot] wants to merge 1 commit into
mainfrom
test_997788515
Open

copybara-service[bot] wants to merge 1 commit into
mainfrom
test_997788515

Conversation

@copybara-service

Copy link
Copy Markdown

fix(dev): default adk.web.cors.origins to same-origin only instead of *

  • Applies to the REST API and the /run_live WebSocket. The bundled dev UI is same-origin and needs no configuration.
  • A UI served from another origin now needs adk.web.cors.origins, for example adk.web.cors.origins=http://localhost:4200.
  • Behind a reverse proxy, a request whose origin matches the URL in the proxy's Forwarded header, or its X-Forwarded-* headers when Forwarded is absent, counts as same-origin. A proxy that hides the external scheme, host or non-default port needs adk.web.cors.origins.

…of `*`

- Applies to the REST API and the `/run_live` WebSocket. The bundled dev UI is same-origin and needs no configuration.
- A UI served from another origin now needs `adk.web.cors.origins`, for example `adk.web.cors.origins=http://localhost:4200`.
- Behind a reverse proxy, a request whose origin matches the URL in the proxy's `Forwarded` header, or its `X-Forwarded-*` headers when `Forwarded` is absent, counts as same-origin. A proxy that hides the external scheme, host or non-default port needs `adk.web.cors.origins`.

PiperOrigin-RevId: 997788515
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.

0 participants