Skip to content

Add missing env_delim to ALLOWED_HOSTS parameter#1395

Merged
umputun merged 1 commit intomasterfrom
paskal/allowed_hosts
Jun 30, 2022
Merged

Add missing env_delim to ALLOWED_HOSTS parameter#1395
umputun merged 1 commit intomasterfrom
paskal/allowed_hosts

Conversation

@paskal
Copy link
Copy Markdown
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.

2 participants