ipsilon

Clone
Source Code
GIT

#136 Do not crash if FAS didn't have an SSH key for the user
Merged 7 years ago by puiterwijk. Opened 7 years ago by puiterwijk.
puiterwijk/ipsilon fas-dontrequiressh  into  master

file modified
+2 -1 
@@ -42,7 +42,8 @@ 
 

      userdata, fas_extra = fas_mapper.map_attributes(fas_data)
 

  

      # We need to split ssh keys by newline, since we can't send newlines
 

-     userdata['ssh_key'] = userdata['ssh_key'].split('\n')
 

+     if userdata.get('ssh_key'):
 

+         userdata['ssh_key'] = userdata['ssh_key'].split('\n')
 

  

      # compute and store groups and cla groups
 

      userdata['_groups'] = []

Signed-off-by: Patrick Uiterwijk puiterwijk@redhat.com

It is safer to use: if userdata.get('ssh_key'):

True.

rebased
7 years ago

Looks fine.

rebased
7 years ago

Thanks @rcritten and @merlinthp!

Pull-Request has been merged by puiterwijk
7 years ago
Metadata
None
No Tags
Changes Summary 1
+2 -1
file changed
ipsilon/info/infofas.py
Powered by Pagure 5.13.3
Documentation • File an Issue • About • SSH Hostkey/Fingerprint
© Red Hat, Inc. and others.