Skip to content

fix(domain): trust current proxy host instead of cookie - #1

Draft
Alidantech wants to merge 1 commit into
mainfrom
chatgpt/secure-domain-resolution
Draft

fix(domain): trust current proxy host instead of cookie#1
Alidantech wants to merge 1 commit into
mainfrom
chatgpt/secure-domain-resolution

Conversation

@Alidantech

Copy link
Copy Markdown
Contributor

Summary

  • stop using the caller-supplied client_url cookie as domain authority
  • derive x-client-url from the current request's proxy-owned forwarding headers
  • normalize and validate the forwarded host before sending backend requests

Security impact

Prevents a stale or forged first-request cookie from selecting another tenant/domain.

Files

  • src/client.ts

Validation

  • branch diff reviewed
  • no .github/** files changed
  • no secrets or generated caches changed

Release note

Publish a new package version and refresh consuming lockfiles before production deployment.

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