Fuel CLI allows to set empty password

Bug #1349734 reported by Łukasz Oleś
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
High
Kamil Sambor

Bug Description

Running
fuel user --change-password

sets empty password for user admin and user is not able to login to GUI or to change password back.

The only way to fix it is to reset password with keystone command using admin_token.

Fuel CLI should not allow setting empty password.

Changed in fuel:
importance: Undecided → Medium
Revision history for this message
Aleksey Kasatkin (alekseyk-ru) wrote :

I changed password back with "fuel user --change-password".
I couldn't repeat your situation when user is not able to login to GUI. It didn't show me "login" window. It was newly deployed master node. It seems that some info is cached in browser and UI don't check that password was not entered because of that.

Revision history for this message
Aleksey Kasatkin (alekseyk-ru) wrote :

Vitaliy, please check why UI doesn't ask password after node is redeployed. How can it be solved?

Changed in fuel:
assignee: Fuel Python Team (fuel-python) → Fuel UI Team (fuel-ui)
Łukasz Oleś (loles)
description: updated
Revision history for this message
Vitaly Kramskikh (vkramskikh) wrote :

What issue should be solved? UI remembers login/password pair from the previous login and tries to authenticate using it. If it succeeds, no auth window is displayed. What is wrong here?

Changed in fuel:
assignee: Fuel UI Team (fuel-ui) → Fuel Python Team (fuel-python)
Revision history for this message
Aleksey Kasatkin (alekseyk-ru) wrote :

Sorry. Authorization was switched off on my env.
Actually, after I've changed password with "fuel user --change-password" to anything, not only to empty, it behaves this way. I.e. UI cannot pass authorization and password cannot be changed using CLI.

Changed in fuel:
status: New → Confirmed
importance: Medium → High
milestone: none → 5.1
Kamil Sambor (ksambor)
Changed in fuel:
assignee: Fuel Python Team (fuel-python) → Kamil Sambor (ksambor)
Kamil Sambor (ksambor)
Changed in fuel:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-web (master)

Fix proposed to branch: master
Review: https://review.openstack.org/110330

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (master)

Reviewed: https://review.openstack.org/110330
Committed: https://git.openstack.org/cgit/stackforge/fuel-web/commit/?id=0b28088c002ba9b36e9840b756a66de5ce328914
Submitter: Jenkins
Branch: master

commit 0b28088c002ba9b36e9840b756a66de5ce328914
Author: Kamil Sambor <email address hidden>
Date: Tue Jul 29 17:29:35 2014 +0200

    Added validation of new password during changing in fuel-cli

    Change-Id: I26f1a94d9137b246d248299b4842038ae293b43a
    Closes-Bug: #1349734

Changed in fuel:
status: In Progress → Fix Committed
Kamil Sambor (ksambor)
Changed in fuel:
status: Fix Committed → Fix Released
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.