Allow the ldap logins with email or username

Change-Id: I73b4aa4c46afd56ae02fb24abcd673c0724547e4

Allow the ldap logins with email or username
Change-Id: I73b4aa4c46afd56ae02fb24abcd673c0724547e4
Izaak Alpert
1 parent 2c191ab0
Showing 2 changed files with 9 additions and 1 deletions Show diff stats
config/gitlab.yml.example
config/initializers/devise.rb
@@ -97,6 +97,7 @@ production: &amp;base
     method: 'ssl' # "ssl" or "plain"
     bind_dn: '_the_full_dn_of_the_user_you_will_bind_with'
     password: '_the_password_of_the_bind_user'
+    allow_username_or_email_login: true
   ## OmniAuth settings
   omniauth:
@@ -206,6 +206,12 @@ Devise.setup do |config|
   # end
   if Gitlab.config.ldap.enabled
+    if Gitlab.config.ldap.allow_username_or_email_login
+      email_stripping_proc = Proc.new {|name| name.gsub(/@.*$/,'')}
+    else
+      email_stripping_proc = lambda {|n|n}
+
+    end
     config.omniauth :ldap,
       host:     Gitlab.config.ldap['host'],
       base:     Gitlab.config.ldap['base'],
@@ -213,7 +219,8 @@ Devise.setup do |config|
       port:     Gitlab.config.ldap['port'],
       method:   Gitlab.config.ldap['method'],
       bind_dn:  Gitlab.config.ldap['bind_dn'],
-      password: Gitlab.config.ldap['password']
+      password: Gitlab.config.ldap['password'],
+      name_proc: email_stripping_proc
   end
   Gitlab.config.omniauth.providers.each do |provider|
	@@ -97,6 +97,7 @@ production: &base		@@ -97,6 +97,7 @@ production: &base
97	method: 'ssl' # "ssl" or "plain"	97	method: 'ssl' # "ssl" or "plain"
98	bind_dn: '_the_full_dn_of_the_user_you_will_bind_with'	98	bind_dn: '_the_full_dn_of_the_user_you_will_bind_with'
99	password: '_the_password_of_the_bind_user'	99	password: '_the_password_of_the_bind_user'
		100	+ allow_username_or_email_login: true
100		101
101	## OmniAuth settings	102	## OmniAuth settings
102	omniauth:	103	omniauth: