summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDouwe Maan <douwe@gitlab.com>2015-04-06 13:26:41 +0200
committerDouwe Maan <douwe@gitlab.com>2015-04-06 13:27:19 +0200
commitf2af30f19534c639c17fa5356ab1c18f518f2e98 (patch)
treef7aa6556d235244308b9c0645c1bef189982fa40
parent45ca39e81bf857cdb58b1a184b8fab0e56c200f7 (diff)
downloadgitlab-ce-skip-email-reconfirmation.tar.gz
Skip email confirmation when set by admin or via LDAP.skip-email-reconfirmation
-rw-r--r--CHANGELOG1
-rw-r--r--app/controllers/admin/users_controller.rb2
-rw-r--r--lib/gitlab/ldap/user.rb1
3 files changed, 3 insertions, 1 deletions
diff --git a/CHANGELOG b/CHANGELOG
index db3e5744b0e..7d2edfbe352 100644
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -62,6 +62,7 @@ v 7.10.0 (unreleased)
- Project labels are now available over the API under the "tag_list" field (Cristian Medina)
- Fixed link paths for HTTP and SSH on the admin project view (Jeremy Maziarz)
- Fix and improve help rendering (Sullivan Sénéchal)
+ - Skip email confirmation when set by admin or via LDAP.
v 7.9.2
diff --git a/app/controllers/admin/users_controller.rb b/app/controllers/admin/users_controller.rb
index 693970e5349..b4c011f213c 100644
--- a/app/controllers/admin/users_controller.rb
+++ b/app/controllers/admin/users_controller.rb
@@ -72,8 +72,8 @@ class Admin::UsersController < Admin::ApplicationController
end
respond_to do |format|
+ user.skip_reconfirmation!
if user.update_attributes(user_params_with_pass)
- user.confirm!
format.html { redirect_to [:admin, user], notice: 'User was successfully updated.' }
format.json { head :ok }
else
diff --git a/lib/gitlab/ldap/user.rb b/lib/gitlab/ldap/user.rb
index cfa8692659d..806f43c4694 100644
--- a/lib/gitlab/ldap/user.rb
+++ b/lib/gitlab/ldap/user.rb
@@ -39,6 +39,7 @@ module Gitlab
end
def update_user_attributes
+ gl_user.skip_reconfirmation!
gl_user.email = auth_hash.email
# Build new identity only if we dont have have same one