Hide LDAP login if not configured #89 (#91)

betadots · Oct 18, 2022 · e01ae92 · e01ae92
1 parent 4e6f6b5
commit e01ae92
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 8 deletions.
diff --git a/app/models/ldap.rb b/app/models/ldap.rb
@@ -1,4 +1,8 @@
 class Ldap
+  def self.configured?
+    Rails.configuration.hdm[:ldap].present?
+  end
+
   def initialize
     hdm_ldap_config = Rails.configuration.hdm[:ldap]
     @host = hdm_ldap_config[:host]

diff --git a/app/views/sessions/new.html.erb b/app/views/sessions/new.html.erb
@@ -1,13 +1,15 @@
 <h1>Login</h1>
 
-<ul class="nav nav-tabs">
-  <li class="nav-item">
-    <%= link_to "Local login", login_path, class: "nav-link active" %>
-  </li>
-  <li class="nav-item">
-    <%= link_to "LDAP login", new_ldap_session_path, class: "nav-link" %>
-  </li>
-</ul>
+<% if Ldap.configured? %>
+  <ul class="nav nav-tabs">
+    <li class="nav-item">
+      <%= link_to "Local login", login_path, class: "nav-link active" %>
+    </li>
+    <li class="nav-item">
+      <%= link_to "LDAP login", new_ldap_session_path, class: "nav-link" %>
+    </li>
+  </ul>
+<% end %>
 
 <%= form_tag sessions_path do |form| %>
   <div class="form-group">