Infrastructure/ipsilon-fedora
1
0
Fork 0
Code Issues 1 Pull requests Projects Releases Packages Wiki Activity Actions

fix infofasjson get_user_attrs returning None

Browse source 
Previously if using the fasjson info plugin, and the fasjson client
returned an error, we were returning None. However, we need to return an
empty dict, otherwise login/common.py in ipsilon would raise an error.

Signed-off-by: Ryan Lerch <rlerch@redhat.com>
This commit is contained in:
Ryan Lercho 2020-09-16 21:37:35 +10:00
parent d7da4d53da
commit 191f2f0fcf
1 changed files with 1 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

2
ipsilon/info/infofasjson.py
View file

@ -48,7 +48,7 @@ Info plugin that retrieves user data from FASJSON. """
user_agreements_data = client.list_user_agreements(username=user).result user_agreements_data = client.list_user_agreements(username=user).result
except Exception as e: except Exception as e:
self.error(f'FASJSON error: {e}') self.error(f'FASJSON error: {e}')
return return {}
# assumption that first email is the default # assumption that first email is the default
user_data['email'] = user_data['emails'][0] user_data['email'] = user_data['emails'][0]