Manually merge PR166 to add auto sign up for externally authenticated users with active peer acceptance even if they don't fit the direct auto sign up filters.

git-svn-id: svn+ssh://svn.code.sf.net/p/migrid/code/trunk@6180 b75ad72c-e7d7-11dd-a971-7dbc132099af

Author: jonasbardino

mig/install/MiGserver-template.conf

@@ -30,6 +30,12 @@ auto_add_filter_method = __AUTO_ADD_FILTER_METHOD__
 # auth methods explicitly enabled with auto_add_X_user. Space separated list of
 # user field and regexp-filter pattern pairs separated by colons.
 auto_add_user_permit = __AUTO_ADD_USER_PERMIT__
+# Optional limit on users who may sign up through autocreate without operator
+# interaction if a valid peer exists. Defaults to allow ANY distinguished name
+# if unset but only for auth methods explicitly enabled with auto_add_X_user.
+# Space separated list of user field and regexp-filter pattern pairs separated
+# by colons.
+auto_add_user_with_peer = __AUTO_ADD_USER_WITH_PEER__
 # Default account expiry unless set. Renew and web login extends by default.
 cert_valid_days = __CERT_VALID_DAYS__
 oid_valid_days = __OID_VALID_DAYS__

mig/install/generateconfs.py

@@ -73,6 +73,7 @@ def main(argv, _generate_confs=generate_confs, _print=print):
         'auto_add_filter_fields',
         'auto_add_filter_method',
         'auto_add_user_permit',
+        'auto_add_user_with_peer',
         'base_fqdn',
         'public_fqdn',
         'public_alias_fqdn',

mig/shared/accountreq.py

@@ -1244,17 +1244,41 @@ def user_manage_commands(configuration, mig_user, req_path, user_id, user_dict,
     return cmd_helpers
 
 
-def auto_add_user_allowed(configuration, user_dict):
+def __auto_add_user_allowed(configuration, user_dict, permit_list):
     """Check if user with user_dict is allowed to sign up without operator
-    approval e.g. using autocreate based on optional configuration limits.
+    approval e.g. using autocreate based on optional configuration limits in
+    given permit_list of fields and regex values. Always fail if permit_list
+    is empty.
     """
 
-    for (key, val) in configuration.auto_add_user_permit:
+    if not permit_list:
+        return False
+    for (key, val) in permit_list:
         if not re.match(val, user_dict.get(key, 'NO SUCH FIELD')):
             return False
     return True
 
 
+def auto_add_user_allowed_direct(configuration, user_dict):
+    """Check if user with user_dict is allowed to sign up directly e.g. using
+    autocreate without operator or peer approval. The check is based on
+    optional configuration limits and must match all such permit expressions.
+    """
+    return __auto_add_user_allowed(configuration, user_dict,
+                                   configuration.auto_add_user_permit)
+
+
+def auto_add_user_allowed_with_peer(configuration, user_dict):
+    """Check if user with user_dict is allowed to sign up with peer acceptance
+    e.g. using autocreate without explicit operator approval. The check is
+    based on optional configuration limits and must match all such permit
+    expressions.
+    """
+
+    return __auto_add_user_allowed(configuration, user_dict,
+                                   configuration.auto_add_user_with_peer)
+
+
 def peers_permit_allowed(configuration, user_dict):
     """Check if user with user_dict is allowed to manage peers based on
     optional configuration limits.

mig/shared/configuration.py

@@ -151,6 +151,7 @@ def fix_missing(config_file, verbose=True):
         'auto_add_oidc_user': False,
         'auto_add_resource': False,
         'auto_add_user_permit': 'distinguished_name:.*',
+        'auto_add_user_with_peer': 'distinguished_name:.*',
         'auto_add_filter_method': '',
         'auto_add_filter_fields': '',
         'server_fqdn': fqdn,
@@ -693,6 +694,7 @@ def get(self, *args, **kwargs):
     'auto_add_oidc_user': False,
     'auto_add_resource': False,
     'auto_add_user_permit': [('distinguished_name', '.*')],
+    'auto_add_user_with_peer': [('distinguished_name', '.*')],
     'auto_add_filter_method': '',
     'auto_add_filter_fields': [],
 
@@ -2630,12 +2632,21 @@ def reload_config(self, verbose, skip_log=False, disable_auth_log=False,
         if config.has_option('GLOBAL', 'auto_add_resource'):
             self.auto_add_resource = config.getboolean('GLOBAL',
                                                        'auto_add_resource')
-        # Limit sign up without operator interaction using ID fields regex.
+        # Limit direct sign up without operator interaction using ID field and
+        # regex pairs.
         # For autocreate auto_add_X_user must be True and auto_add_user_permit
         # specification must match actual user on all given fields.
         if config.has_option('GLOBAL', 'auto_add_user_permit'):
             req = config.get('GLOBAL', 'auto_add_user_permit').split()
             self.auto_add_user_permit = [i.split(':', 2) for i in req]
+        # Limit peer accepted sign up without operator interaction using ID
+        # field and regex pairs.
+        # For autocreate auto_add_X_user must be True and
+        # auto_add_user_with_peer specification must match actual user on all
+        # given fields. Plus an active peer acceptance to match must exist.
+        if config.has_option('GLOBAL', 'auto_add_user_with_peer'):
+            req = config.get('GLOBAL', 'auto_add_user_with_peer').split()
+            self.auto_add_user_with_peer = [i.split(':', 2) for i in req]
 
         # Apply requested automatic filtering of selected auto add user fields
         if config.has_option('GLOBAL', 'auto_add_filter_method'):

mig/shared/functionality/autocreate.py

@@ -33,7 +33,7 @@
    Also see req-/extcertaction.py
    Differences:
      - automatic upload of a proxy certificate when provided
-     - no special check for KU organization
+     - no special check for organization and email match
      - allows empty fields for things like country, email, and state
 """
 
@@ -46,7 +46,8 @@
 import time
 
 from mig.shared import returnvalues
-from mig.shared.accountreq import auto_add_user_allowed
+from mig.shared.accountreq import auto_add_user_allowed_direct, \
+    auto_add_user_allowed_with_peer
 from mig.shared.accountstate import default_account_expire
 from mig.shared.bailout import filter_output_objects
 from mig.shared.base import client_id_dir, canonical_user, mask_creds, \
@@ -729,7 +730,13 @@ def main(client_id, user_arguments_dict, environ=None):
             configuration.auto_add_oidc_user:
         fill_user(user_dict)
 
-        if not auto_add_user_allowed(configuration, user_dict):
+        if auto_add_user_allowed_direct(configuration, user_dict):
+            logger.debug('autocreate directly permitted for %s' % client_id)
+        elif auto_add_user_allowed_with_peer(configuration, user_dict):
+            logger.debug('autocreate only permitted with peer for %s' %
+                         client_id)
+            peer_pattern = keyword_auto
+        else:
             logger.warning('autocreate not permitted for %s' % client_id)
             output_objects.append({
                 'object_type': 'error_text', 'text':

mig/shared/install.py

@@ -331,6 +331,7 @@ def generate_confs(
     auto_add_filter_fields='',
     auto_add_filter_method='skip',
     auto_add_user_permit='distinguished_name:.*',
+    auto_add_user_with_peer='distinguished_name:.*',
     cert_valid_days=365,
     oid_valid_days=365,
     oidc_valid_days=365,
@@ -648,6 +649,7 @@ def _generate_confs_prepare(
     auto_add_filter_fields,
     auto_add_filter_method,
     auto_add_user_permit,
+    auto_add_user_with_peer,
     cert_valid_days,
     oid_valid_days,
     oidc_valid_days,
@@ -895,6 +897,7 @@ def _generate_confs_prepare(
     user_dict['__AUTO_ADD_FILTER_FIELDS__'] = auto_add_filter_fields
     user_dict['__AUTO_ADD_FILTER_METHOD__'] = auto_add_filter_method
     user_dict['__AUTO_ADD_USER_PERMIT__'] = auto_add_user_permit
+    user_dict['__AUTO_ADD_USER_WITH_PEER__'] = auto_add_user_with_peer
     user_dict['__CERT_VALID_DAYS__'] = "%s" % cert_valid_days
     user_dict['__OID_VALID_DAYS__'] = "%s" % oid_valid_days
     user_dict['__OIDC_VALID_DAYS__'] = "%s" % oidc_valid_days
@@ -2596,6 +2599,7 @@ def create_user(
     auto_add_filter_fields = ''
     auto_add_filter_method = 'skip'
     auto_add_user_permit = 'distinguished_name:.*'
+    auto_add_user_with_peer = 'distinguished_name:.*'
     cert_valid_days = 365
     oid_valid_days = 365
     oidc_valid_days = 365