Bug 782075

Summary: Invalid SSH key error message for ssh keys with space in description
Product: [Retired] Beaker Reporter: Kamil Páral <kparal>
Component: web UIAssignee: Steven Lawrance <stl>
Status: CLOSED CURRENTRELEASE QA Contact:
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 0.8CC: bpeck, dcallagh, mcsontos, rmancy, stl
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-04-26 07:16:31 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Attachments:
Description Flags
key.pub none

Description Kamil Páral 2012-01-16 13:15:02 UTC 
Description of problem:
In user preferences I tried to add this public ssh key:

ssh-rsa AAAAB3NzaC1yc2EAAAADAQABAAABAQCosl4BAGt1H+04bHc8i7kFtftxA/JxdM7Gm6uLhGRjgw6r8eAI3anv50aLZKLJcGFYBpFyMTdscfzLTBSNpRIJj1zGfXY7oXvdQswV3UidiEkm5nVYuVSD7lvNqgM4sr4ZBNOj4SDp0u6hEQSejUGwYmtD/3OhS+9NB4+viD17jiSu7pzqI8qyJPqJ3OAykyDokA6S8VtAFNxo37ntt+HgbpjRCGUCld4FLbjZCMfmo4tNHjyptjhk/PhuNK5N1o4XqH9ECM7AwEI7Ou4p5YUAJSQb90Z2ELmQByIGEwStHXBhSJdAW7OgY2aONeSU/5y+9oaOX7G3sgA/u1tWzhu/ kparal@venus (master)

Notice there is a space in key description. Beaker says "Invalid SSH key".

This key, however, can be added:

ssh-rsa AAAAB3NzaC1yc2EAAAADAQABAAABAQCosl4BAGt1H+04bHc8i7kFtftxA/JxdM7Gm6uLhGRjgw6r8eAI3anv50aLZKLJcGFYBpFyMTdscfzLTBSNpRIJj1zGfXY7oXvdQswV3UidiEkm5nVYuVSD7lvNqgM4sr4ZBNOj4SDp0u6hEQSejUGwYmtD/3OhS+9NB4+viD17jiSu7pzqI8qyJPqJ3OAykyDokA6S8VtAFNxo37ntt+HgbpjRCGUCld4FLbjZCMfmo4tNHjyptjhk/PhuNK5N1o4XqH9ECM7AwEI7Ou4p5YUAJSQb90Z2ELmQByIGEwStHXBhSJdAW7OgY2aONeSU/5y+9oaOX7G3sgA/u1tWzhu/ kparal@venus(master)

That seems like a bug in Beaker.
Comment 1 Kamil Páral 2012-01-16 13:18:02 UTC 
Created attachment 555502 [details]
key.pub

Uploading the key because of text wrapping issues here.
Comment 2 Steven Lawrance 2012-01-27 05:44:46 UTC 
Couldn't find any definitive answer on what is/isn't allowed in the description so I've just changed the split() call to use maxsplit=2.