From b7fc7999c3b64ec5e59ec1b4ea58c6339104a947 Mon Sep 17 00:00:00 2001
From: Joseph Schorr <josephschorr@users.noreply.github.com>
Date: Thu, 20 Oct 2016 14:58:35 -0400
Subject: [PATCH] Delete old "license" checking code arounds user counts

This is legacy code that doesn't actually do anything of value
---
 data/model/__init__.py    |  4 ----
 data/model/user.py        | 12 +-----------
 endpoints/api/__init__.py |  4 ----
 endpoints/api/user.py     | 12 ++----------
 4 files changed, 3 insertions(+), 29 deletions(-)

diff --git a/data/model/__init__.py b/data/model/__init__.py
index 6c35a6ae9..23a1a647c 100644
--- a/data/model/__init__.py
+++ b/data/model/__init__.py
@@ -45,10 +45,6 @@ class InvalidUsernameException(DataModelException):
   pass
 
 
-class TooManyUsersException(DataModelException):
-  pass
-
-
 class InvalidRepositoryBuildException(DataModelException):
   pass
 
diff --git a/data/model/user.py b/data/model/user.py
index a711efda7..004f3e58a 100644
--- a/data/model/user.py
+++ b/data/model/user.py
@@ -12,7 +12,7 @@ from data.database import (User, LoginService, FederatedLogin, RepositoryPermiss
                            UserRegion, ImageStorageLocation)
 from data.model import (DataModelException, InvalidPasswordException, InvalidRobotException,
                         InvalidUsernameException, InvalidEmailAddressException,
-                        TooManyUsersException, TooManyLoginAttemptsException, db_transaction,
+                        TooManyLoginAttemptsException, db_transaction,
                         notification, config, repository, _basequery)
 from util.names import format_robot_username, parse_robot_username
 from util.validation import (validate_username, validate_email, validate_password,
@@ -31,18 +31,11 @@ def hash_password(password, salt=None):
   return bcrypt.hashpw(password.encode('utf-8'), salt)
 
 
-def is_create_user_allowed():
-  return True
-
-
 def create_user(username, password, email, auto_verify=False):
   """ Creates a regular user, if allowed. """
   if not validate_password(password):
     raise InvalidPasswordException(INVALID_PASSWORD_MESSAGE)
 
-  if not is_create_user_allowed():
-    raise TooManyUsersException()
-
   created = create_user_noverify(username, email)
   created.password_hash = hash_password(password)
   created.verified = auto_verify
@@ -307,9 +300,6 @@ def list_entity_robot_permission_teams(entity_name, include_permissions=False):
 
 def create_federated_user(username, email, service_name, service_ident,
                           set_password_notification, metadata={}):
-  if not is_create_user_allowed():
-    raise TooManyUsersException()
-
   new_user = create_user_noverify(username, email)
   new_user.verified = True
   new_user.save()
diff --git a/endpoints/api/__init__.py b/endpoints/api/__init__.py
index d12600f32..d9c3d0fb0 100644
--- a/endpoints/api/__init__.py
+++ b/endpoints/api/__init__.py
@@ -322,10 +322,6 @@ def request_error(exception=None, **kwargs):
   raise InvalidRequest(message, data)
 
 
-def license_error(exception=None):
-  raise ExceedsLicenseException()
-
-
 def log_action(kind, user_or_orgname, metadata=None, repo=None):
   if not metadata:
     metadata = {}
diff --git a/endpoints/api/user.py b/endpoints/api/user.py
index c6c75c0be..f7f12fd2d 100644
--- a/endpoints/api/user.py
+++ b/endpoints/api/user.py
@@ -21,7 +21,7 @@ from data.database import Repository as RepositoryTable
 from endpoints.api import (ApiResource, nickname, resource, validate_json_request, request_error,
                            log_action, internal_only, require_user_admin, parse_args,
                            query_param, require_scope, format_date, show_if,
-                           license_error, require_fresh_login, path_param, define_json_response,
+                           require_fresh_login, path_param, define_json_response,
                            RepositoryParamResource, page_support)
 from endpoints.exception import NotFound, InvalidToken
 from endpoints.api.subscribe import subscribe
@@ -367,9 +367,6 @@ class User(ApiResource):
       else:
         common_login(new_user)
         return user_view(new_user)
-
-    except model.user.TooManyUsersException as ex:
-      raise license_error(exception=ex)
     except model.user.DataModelException as ex:
       raise request_error(exception=ex)
 
@@ -439,13 +436,8 @@ class ClientKey(ApiResource):
 def conduct_signin(username_or_email, password, invite_code=None):
   needs_email_verification = False
   invalid_credentials = False
-  found_user = None
-
-  try:
-    (found_user, error_message) = authentication.verify_and_link_user(username_or_email, password)
-  except model.user.TooManyUsersException as ex:
-    raise license_error(exception=ex)
 
+  (found_user, error_message) = authentication.verify_and_link_user(username_or_email, password)
   if found_user:
     # If there is an attached invitation code, handle it here. This will mark the
     # user as verified if the code is valid.