Closed mattycourtney closed 3 years ago
I am a bot, here are the test results for this PR: https://ci-tests.linuxserver.io/lspipepr/netbox/v2.10.4-pkg-0a2c9289-pr-13/index.html https://ci-tests.linuxserver.io/lspipepr/netbox/v2.10.4-pkg-0a2c9289-pr-13/shellcheck-result.xml
I'm really keen to see this get merged, as I use REMOTE_AUTH to SSO to Netbox hosted behind an identity aware proxy. Please let me know if there's any more detail I can provide within this PR that will aid your evaluation.
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
bump
I am a bot, here are the test results for this PR: https://ci-tests.linuxserver.io/lspipepr/netbox/v2.10.6-pkg-4cdd4aed-pr-13/index.html https://ci-tests.linuxserver.io/lspipepr/netbox/v2.10.6-pkg-4cdd4aed-pr-13/shellcheck-result.xml
LGTM
Description:
Added environment variables to control NetBox's remote authentication configuration parameters
Benefits of this PR and context:
This PR enables users of the container image to control NetBox's remote authentication configuration parameters using environment variables.
Configuring remote authentication facilitates SSO via reverse proxy servers that add a HTTP header to indicate the authenticated user's identity.
How Has This Been Tested?
REMOTE_AUTH_*
environment variables set, and validated that the generated configuration.py file was as expectedREMOTE_AUTH_*
environment variables set, and validated that the values were present in the configuration.py fileSource / References: