-
-
Notifications
You must be signed in to change notification settings - Fork 893
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Some REMOTE_AUTH_ settings missing #992
Comments
We cannot set up Azure AD (Tenants), the user ends up getting redirected to the Azure auth server with This seems to be caused by this issue. The could be related to #653 |
We are having this issue as well |
Was able to fix by adding the following to SOCIAL_AUTH_AZUREAD_TENANT_OAUTH2_KEY = environ.get('SOCIAL_AUTH_AZUREAD_TENANT_OAUTH2_KEY', '')
SOCIAL_AUTH_AZUREAD_TENANT_OAUTH2_SECRET = environ.get('SOCIAL_AUTH_AZUREAD_TENANT_OAUTH2_SECRET', '')
SOCIAL_AUTH_AZUREAD_TENANT_OAUTH2_TENANT_ID = environ.get('SOCIAL_AUTH_AZUREAD_TENANT_OAUTH2_TENANT_ID', '') Of course, you'll need to add whichever one you're using, I am using tenant. |
I just opened a PR that adds the missing config options #1144 |
Desired Behavior
I'd like to be able to provide all the Remote Auth settings as environment variables.
Contrast to Current Behavior
Currently only
are able to be pulled in from environment variables
Required Changes
Adding the ability to declare the other environment variables.
Discussion: Benefits and Drawbacks
I'm aware that I can do it currently by adding an extra file to the configuration folder but it would be good to not have to mount an extra file into the container.
Also aware that
REMOTE_AUTH_DEFAULT_PERMISSIONS
isn't recommended due to the complexity of the value - ref #890 - but that shouldn't preclude the other variables from being made available.The text was updated successfully, but these errors were encountered: