SSH key page shows import banner

Bug #1859195 reported by Tom Wardill
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Canonical SSO provider
Incomplete
Medium
Karl Williams

Bug Description

Under the vanilla layout, the 'SSH keys' pages shows a banner telling me that I need to set a username before adding a key.

There are two problems with this.
1. I already have set a username, so why does this banner show?
2. If I haven't set a username, why are the controls to add a key enabled?

Tags: vanilla
Revision history for this message
Tom Wardill (twom) wrote :
Daniel Manrique (roadmr)
Changed in canonical-identity-provider:
importance: Undecided → High
Revision history for this message
Karl Williams (deadlight) wrote :

Re: 1., The form generation code passes the "unset username" error to the template and this has not changed with the frontend conversion to using Vanilla. Perhaps this is more of an artefact of the environment? Maybe more testing is needed.

Re: 2., I will make sure that the form submission button is disabled when the rest of the form is. (Perhaps, as a later enhancement, we can consider not showing the form at all if the user cannot add a key)

Changed in canonical-identity-provider:
assignee: nobody → Karl Williams (deadlight)
Revision history for this message
Daniel Manrique (roadmr) wrote :

Hey Tom,

Can you please provide more details on a repro case for this? I wasn't able to on staging, maybe because staging and production force users to have a username.

If that's the case, a repro case maybe with a local install would help, but we can also lower the priority since it wouldn't affect deployed public instances.

Changed in canonical-identity-provider:
status: New → Incomplete
Revision history for this message
Daniel Manrique (roadmr) wrote :

On staging, it does not show the form if you don't have a username setting, and while the error messages show, they do point to what you need to do (set a username) in order to be able to add keys.

Changed in canonical-identity-provider:
importance: High → Medium
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.