Commit dc023faf by Clinton Blackburn

Added the ability to specify additional whitelisted domains for CORS for

the credentials service.

LEARNER-516
parent 59e98eeb
...@@ -109,9 +109,11 @@ CREDENTIALS_FILE_STORAGE_BACKEND: ...@@ -109,9 +109,11 @@ CREDENTIALS_FILE_STORAGE_BACKEND:
DEFAULT_FILE_STORAGE: 'django.core.files.storage.FileSystemStorage' DEFAULT_FILE_STORAGE: 'django.core.files.storage.FileSystemStorage'
# Note: the protocol for CORS whitelist values is necessary for matching the correct origin by nginx # Note: the protocol for CORS whitelist values is necessary for matching the correct origin by nginx
CREDENTIALS_CORS_WHITELIST: CREDENTIALS_CORS_WHITELIST_DEFAULT:
- "http://{{ CREDENTIALS_DOMAIN }}" - "{{ CREDENTIALS_DOMAIN }}"
- "https://{{ CREDENTIALS_DOMAIN }}"
CREDENTIALS_CORS_WHITELIST_EXTRA: []
CREDENTIALS_CORS_WHITELIST: "{{ CREDENTIALS_CORS_WHITELIST_DEFAULT + CREDENTIALS_CORS_WHITELIST_EXTRA }}"
CREDENTIALS_VERSION: "master" CREDENTIALS_VERSION: "master"
CREDENTIALS_REPOS: CREDENTIALS_REPOS:
......
...@@ -17,7 +17,7 @@ upstream credentials_app_server { ...@@ -17,7 +17,7 @@ upstream credentials_app_server {
map $http_origin $cors_header { map $http_origin $cors_header {
default ""; default "";
'~*^({{ CREDENTIALS_CORS_WHITELIST|join('|')|replace('.', '\.') }})$' "$http_origin"; '~*^https?://({{ CREDENTIALS_CORS_WHITELIST|join('|')|replace('.', '\.') }})$' "$http_origin";
} }
server { server {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment