diff options
| author | Simo Sorce <simo@redhat.com> | 2015-02-16 14:04:49 -0500 |
|---|---|---|
| committer | Patrick Uiterwijk <puiterwijk@redhat.com> | 2015-02-24 16:37:38 +0100 |
| commit | 771b8fd095f3bcb922f761d297c62f1a56a997d5 (patch) | |
| tree | a0b588a1135f97abf6ddff141cb461b1fd389685 /ipsilon/login/authfas.py | |
| parent | dd8a2ecf15a7f74e2fe3d8c5ea0ff5e2fed20927 (diff) | |
| download | ipsilon-771b8fd095f3bcb922f761d297c62f1a56a997d5.tar.gz ipsilon-771b8fd095f3bcb922f761d297c62f1a56a997d5.tar.xz ipsilon-771b8fd095f3bcb922f761d297c62f1a56a997d5.zip | |
Prefix userdata hives with _ to avoid conflicts
The main userdata dict contains common attributes, but we add
a sepcial groups list and unmapped extras, as well as indicators
like auth_type.
All these additional attributes are now prefixed by a _ character
so that conflicts with legitimate attributes are improbable.
Signed-off-by: Simo Sorce <simo@redhat.com>
Reviewed-by: Patrick Uiterwijk <puiterwijk@redhat.com>
Diffstat (limited to 'ipsilon/login/authfas.py')
| -rw-r--r-- | ipsilon/login/authfas.py | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/ipsilon/login/authfas.py b/ipsilon/login/authfas.py index 33d1ac5..3697c1a 100644 --- a/ipsilon/login/authfas.py +++ b/ipsilon/login/authfas.py @@ -82,18 +82,19 @@ class FAS(LoginFormBase): userdata, fas_extra = self.mapper.map_attrs(fas_data) # compute and store groups and cla groups - userdata['groups'] = [] - userdata['extras'] = {'fas': fas_extra, 'cla': []} + userdata['_groups'] = [] + userdata['_extras'] = {'fas': fas_extra, 'cla': []} for group in fas_data.get('approved_memberships', {}): if 'name' not in group: continue if group.get('group_type') == 'cla': if group['name'] in CLA_GROUPS: - userdata['extras']['cla'].append(CLA_GROUPS[group['name']]) + group_name = CLA_GROUPS[group['name']] else: - userdata['extras']['cla'].append(group['name']) + group_name = group['name'] + userdata['_extras']['cla'].append(group_name) else: - userdata['groups'].append(group['name']) + userdata['_groups'].append(group['name']) return userdata |