Skip to content
Snippets Groups Projects
Unverified Commit 3d1f5fa3 authored by David Taylor's avatar David Taylor Committed by GitHub
Browse files

FIX: Hide settings if configured via environment variables (#65)

`name` is nil... so this logic was not working at all
parent f6279c56
No related branches found
No related tags found
No related merge requests found
......@@ -50,7 +50,7 @@ module ::DiscourseSaml
end
after_initialize do
if !!GlobalSetting.try("#{name}_target_url")
if !!GlobalSetting.try("saml_target_url")
# Configured via environment variables. Hide all the site settings
# from the UI to avoid confusion
SiteSetting.defaults.all.keys.each do |k|
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment