ldap: don't crash on search when guessing an email

The `search` method from Net::LDAP might return nil. In this case simply return early as we do on other cases. See #1832 Signed-off-by: Miquel Sabaté Solà <[email protected]>
SUSE · May 28, 2018 · 93df51c · 93df51c
1 parent 45814ba
commit 93df51c
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/lib/portus/ldap.rb b/lib/portus/ldap.rb
@@ -232,7 +232,7 @@ def guess_email
       return nil if cfg.nil? || !cfg["enabled"]
 
       record = @ldap.search(search_options)
-      return nil if record.size != 1
+      return nil if record.blank? || record.size != 1
       record = record.first
 
       cfg["attr"].empty? ? guess_from_dn(record["dn"]) : guess_from_attr(record, cfg["attr"])