Skip to content

E
edx-platform
Commit ed2cba2e by Omar Khan
Options

Keep SAML configuration check

parent 9b7bb0dc
Showing with 2 additions and 1 deletions
common/djangoapps/third_party_auth/saml.py
...@@ -35,10 +35,11 @@ class SAMLAuthBackend(SAMLAuth): # pylint: disable=abstract-method ...@@ -35,10 +35,11 @@ class SAMLAuthBackend(SAMLAuth): # pylint: disable=abstract-method
parameter before getting the URL to which we must redirect in order to parameter before getting the URL to which we must redirect in order to
authenticate the user. authenticate the user.
raise Http404 if SAML is disabled raise Http404 if SAML authentication is disabled.
raise AuthMissingParameter if the 'idp' parameter is missing. raise AuthMissingParameter if the 'idp' parameter is missing.
""" """
if not self._config.enabled: if not self._config.enabled:
log.error('SAML authentication is not enabled')
raise Http404 raise Http404
# TODO: remove this check once the fix is merged upstream: # TODO: remove this check once the fix is merged upstream:
# https://github.com/omab/python-social-auth/pull/821 # https://github.com/omab/python-social-auth/pull/821
......
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