Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ALLOWED_HOSTS instructions incorrect #4319

Merged
merged 3 commits into from
Jan 9, 2025

Conversation

OriPekelman
Copy link
Contributor

By default, Django's domain name pattern matching in ALLOWED_HOSTS only supports single-level wildcard subdomains (e.g., .example.com). Multiple levels of wildcards (like ..example.com) are not supported. as such '.platformsh.site' will not match '.us-2.platformsh.site'. Users may also set the ALLOWED_HOSTS to 'us-2.platformsh.site'. but that would require specifying the region in the configuration.

Why

Closes #{ISSUE_NUMBER}

What's changed

Where are changes

Updates are for:

  • platform (sites/platform templates)
  • upsun (sites/upsun templates)

By default, Django's domain name pattern matching in ALLOWED_HOSTS only supports single-level wildcard subdomains (e.g., *.example.com). Multiple levels of wildcards (like *.*.example.com) are not supported. as such '.platformsh.site' will not match '*.us-2.platformsh.site'. Users may also set the ALLOWED_HOSTS to 'us-2.platformsh.site'. but that would require specifying the region in the configuration.
@chadwcarlson chadwcarlson added the build-fork To build PRs from forks label Jan 9, 2025
@chadwcarlson chadwcarlson merged commit 2a52a5f into platformsh:main Jan 9, 2025
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build-fork To build PRs from forks markdown/prose
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants