kallithea Changeset - 6944b1249f28

Changeset - 6944b1249f28

Parent rev.

Child rev.

[Not reviewed]

beta

0 2 0

Marcin Kuzminski - 14 years ago 2011-11-16 20:31:14
marcin@python-works.com

fixed issues with not unique emails when using ldap or container auth.

2 files changed with 6 insertions and 3 deletions:

rhodecode/lib/auth_ldap.py

rhodecode/model/user.py

0 comments (0 inline, 0 general)

rhodecode/lib/auth_ldap.py

➞

Show inline comments

@@ @@ -43,8 +43,7 @@ class AuthLdap(object): @@
     def __init__(self, server, base_dn, port=389, bind_dn='', bind_pass='',
                  tls_kind='PLAIN', tls_reqcert='DEMAND', ldap_version=3,
                  ldap_filter='(&(objectClass=user)(!(objectClass=computer)))',
                  search_scope='SUBTREE',
                  attr_login='uid'):
                  search_scope = 'SUBTREE', attr_login = 'uid'):
         self.ldap_version = ldap_version
         ldap_server_type = 'ldap'

rhodecode/model/user.py

➞

Show inline comments

@@ @@ -149,6 +149,10 @@ class UserModel(BaseModel): @@
         :param attrs:
         """
         if self.get_by_username(username, case_insensitive=True) is None:
             # autogenerate email for container account without one
             generate_email = lambda usr: '%s@container_auth.account' % usr
             try:
                 new_user = User()
                 new_user.username = username
@@ @@ -156,7 +160,7 @@ class UserModel(BaseModel): @@
                 new_user.api_key = generate_api_key(username)
                 new_user.email = attrs['email']
                 new_user.active = attrs.get('active', True)
                 new_user.name = attrs['name']
+                new_user.name = attrs['name'] or generate_email(username)
                 new_user.lastname = attrs['lastname']
                 self.sa.add(new_user)

0 comments (0 inline, 0 general)