Andal.LND/edx-platform
4
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Introduce LMS_INTERNAL_ROOT_URL.

Browse Source 
The LMS and edx-enterprise currently communicate via REST APIs, even
though they are all running within the same service.

Currently API URLs are constructed using LMS_ROOT_URL, but that is
problematic on some systems where the external LMS_ROOT_URL is not
accessible from the private network, or routing internal requests via
the external URL is not desireable.

This patch introduces a new setting variable LMS_INTERNAL_ROOT_URL which
defaults to the value of LMS_ROOT_URL for backwards compatibility.

The LMS and edx-enterprise will use LMS_INTERNAL_ROOT_URL to construct
API URLs instead of LMS_ROOT_URL.
This commit is contained in:
Matjaz Gregoric
2017-11-08 10:15:35 +01:00
parent 705aac7980
commit d403e5f8b1
7 changed files with 22 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
lms/envs/devstack.py
View File

@@ -20,7 +20,8 @@ CELERY_ALWAYS_EAGER = True
HTTPS = 'off'
LMS_ROOT_URL = 'http://localhost:8000'
ENTERPRISE_API_URL = LMS_ROOT_URL + '/enterprise/api/v1/'
LMS_INTERNAL_ROOT_URL = LMS_ROOT_URL
ENTERPRISE_API_URL = LMS_INTERNAL_ROOT_URL + '/enterprise/api/v1/'
################################ LOGGERS ######################################