Bug 782075 - Invalid SSH key error message for ssh keys with space in description
Summary: Invalid SSH key error message for ssh keys with space in description
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Beaker
Classification: Retired
Component: web UI
Version: 0.8
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified vote
Target Milestone: ---
Assignee: Steven Lawrance
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-01-16 13:15 UTC by Kamil Páral
Modified: 2019-05-22 13:41 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-04-26 07:16:31 UTC

Attachments (Terms of Use)
key.pub (403 bytes, application/octet-stream)
2012-01-16 13:18 UTC, Kamil Páral 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

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.
Note You need to log in before you can comment on or make changes to this bug.