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

Add missing env_delim to ALLOWED_HOSTS parameter #1395

Merged
merged 1 commit into from
Jun 30, 2022

Conversation

paskal
Copy link
Sponsor Collaborator

@paskal paskal commented Jun 30, 2022

Fix for a bug potentially affecting #1139.

Before:

# with ALLOWED_HOSTS=domain1 domain2
# header
Content-Security-Policy: frame-ancestors domain1 domain2;
# with ALLOWED_HOSTS=domain1,domain2
# header
Content-Security-Policy: frame-ancestors domain1,domain2;
# console
Unrecognized Content-Security-Policy directive 'domain2'.

After:

# with ALLOWED_HOSTS=domain1,domain2 or ALLOWED_HOSTS=domain1 domain2
# header
Content-Security-Policy: frame-ancestors domain1 domain2;

@paskal paskal requested a review from umputun as a code owner June 30, 2022 03:05
@umputun umputun merged commit 6a5c5a4 into master Jun 30, 2022
@umputun umputun deleted the paskal/allowed_hosts branch June 30, 2022 03:32
@paskal paskal added this to the v1.10.2 milestone Jul 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants