Merge branch 'namespace-validation' into 'master'

Check all namespaces on validation of new username. Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/24519 and https://gitlab.com/gitlab-org/gitlab-ce/issues/24580 See merge request !7537

Merge branch 'namespace-validation' into 'master'
e252ee5b · Rémy Coutable · 492ead3f · ed1cd988 · e252ee5b · e252ee5b
Commit e252ee5b authored Nov 18, 2016 by Rémy Coutable
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 1 deletion

users_controller.rb app/controllers/users_controller.rb +1 -1

namespace-validation.yml changelogs/unreleased/namespace-validation.yml +4 -0

No files found.
--- a/app/controllers/users_controller.rb
+++ b/app/controllers/users_controller.rb
@@ -86,7 +86,7 @@ class UsersController < ApplicationController
  end

  def exists
-    render json: { exists: Namespace.where(path: params[:username].downcase).any? }
+    render json: { exists: !!Namespace.find_by_path_or_name(params[:username]) }
  end

  private

--- a/changelogs/unreleased/namespace-validation.yml
+++ b/changelogs/unreleased/namespace-validation.yml
+---
+title: Check all namespaces on validation of new username.
+merge_request: 7537
+author: