christianlight/tutor
6
0
Fork 0
mirror of https://github.com/ChristianLight/tutor.git synced 2024-12-12 14:17:46 +00:00
Code Issues Releases Activity

Fix oauth authentication in dev mode

Browse Source 
This was causing issues with the discovery service in development mode.
This commit is contained in:
Régis Behmo 2020-01-10 13:14:55 +01:00
parent 899e4dfb9a
commit c636a39af9
4 changed files with 6 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
CHANGELOG.md
View File

@ -4,6 +4,7 @@ Note: Breaking changes between versions are indicated by "💥".
## Unreleased ## Unreleased
- [Bugfix] Fix oauth authentication in dev mode
- [Improvement] Upgrade to the 3.7 docker-compose syntax - [Improvement] Upgrade to the 3.7 docker-compose syntax
- [Improvement] The `dev runserver` command can now be run for just any service - [Improvement] The `dev runserver` command can now be run for just any service
- 💥[Feature] `dev run/exec` commands now support generic options which are passed to docker-compose. Consequently, defining the `TUTOR_EDX_PLATFORM_PATH` environment variable no longer works. Instead, users are encouraged to explicitely pass the `-v` option or define a command alias - 💥[Feature] `dev run/exec` commands now support generic options which are passed to docker-compose. Consequently, defining the `TUTOR_EDX_PLATFORM_PATH` environment variable no longer works. Instead, users are encouraged to explicitely pass the `-v` option or define a command alias

2
tutor/templates/apps/openedx/settings/lms/development.py
View File

@ -4,6 +4,8 @@ from lms.envs.devstack import *
{% include "apps/openedx/settings/partials/common_lms.py" %} {% include "apps/openedx/settings/partials/common_lms.py" %}
OAUTH_OIDC_ISSUER = "{{ JWT_COMMON_ISSUER }}"
# Setup correct webpack configuration file for development # Setup correct webpack configuration file for development
WEBPACK_CONFIG_PATH = "webpack.dev.config.js" WEBPACK_CONFIG_PATH = "webpack.dev.config.js"

3
tutor/templates/apps/openedx/settings/lms/production.py
View File

@ -16,7 +16,4 @@ ALLOWED_HOSTS = [
# Required to display all courses on start page # Required to display all courses on start page
SEARCH_SKIP_ENROLLMENT_START_DATE_FILTERING = True SEARCH_SKIP_ENROLLMENT_START_DATE_FILTERING = True
# Allow insecure oauth2 for local interaction with local containers
OAUTH_ENFORCE_SECURE = False
{{ patch("openedx-lms-production-settings") }} {{ patch("openedx-lms-production-settings") }}

3
tutor/templates/apps/openedx/settings/partials/common_lms.py
View File

@ -30,6 +30,9 @@ JWT_AUTH["JWT_AUDIENCE"] = "{{ JWT_COMMON_AUDIENCE }}"
JWT_AUTH["JWT_SECRET_KEY"] = "{{ JWT_COMMON_SECRET_KEY }}" JWT_AUTH["JWT_SECRET_KEY"] = "{{ JWT_COMMON_SECRET_KEY }}"
JWT_AUTH["JWT_PRIVATE_SIGNING_JWK"] = None JWT_AUTH["JWT_PRIVATE_SIGNING_JWK"] = None
# Allow insecure oauth2 for local interaction with local containers
OAUTH_ENFORCE_SECURE = False
# Create folders if necessary # Create folders if necessary
for folder in [LOG_DIR, MEDIA_ROOT, STATIC_ROOT_BASE, ORA2_FILEUPLOAD_ROOT]: for folder in [LOG_DIR, MEDIA_ROOT, STATIC_ROOT_BASE, ORA2_FILEUPLOAD_ROOT]:
if not os.path.exists(folder): if not os.path.exists(folder):