keystone user CRUD uses "password" instead of "OS-KSADM:password"

Bug #1226466 reported by Alvin Garcia
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Identity (keystone)
Fix Released
Low
David Stanek

Bug Description

When logging in with a user created with "OS-KSADM:password" instead of "password" attribute returns:

{"error": {
    "message": "An unexpected error prevented the server from fulfilling your request. no hash specified",
    "code": 500,
    "title": "Internal Server Error"
  }
}

The same error when logging in with a user created without the "password" or "OS-KSADM:password" attribute.

Ref: http://docs.openstack.org/api/openstack-identity-service/2.0/content/POST_addUser_v2.0_users_.html

tags: added: api user
Revision history for this message
Dolph Mathews (dolph) wrote :

The first part of this was a dupe of bug 1214686; truncated the description accordingly.

Changed in keystone:
importance: Undecided → Low
summary: - Add user on v2.0 does not match documentation specs
+ keystone user CRUD uses "password" instead of "OS-KSADM:password"
description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to keystone (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to keystone (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/47930

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to keystone (master)

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

Changed in keystone:
assignee: nobody → David Stanek (dstanek)
status: New → In Progress
Dolph Mathews (dolph)
tags: added: havana-backport-potential
Revision history for this message
David Stanek (dstanek) wrote :

I added some functionality to deal with the password problems and thats not in review (https://review.openstack.org/#/c/48575/).

I'm also working on the OS-KSADM:password functionality. I've refactored some of the XML serialization to make writing tests for this a little easier.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to keystone (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/49062

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

Reviewed: https://review.openstack.org/48575
Committed: http://github.com/openstack/keystone/commit/dc48eee9d2519191c58d97fc48e3881925897275
Submitter: Jenkins
Branch: master

commit dc48eee9d2519191c58d97fc48e3881925897275
Author: David Stanek <email address hidden>
Date: Thu Sep 26 23:54:49 2013 +0000

    Fixes errors logging in as a user with no password

    A traceback in passlibe happened when trying to login as a user that
    didn't have a password set. This gave an unexpected status code. This
    patch makes the server return a 401 instead.

    Change-Id: I9cc64aeefb9e7488579482d4ee95df198af70801
    Partial-Bug: #1226466

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to keystone (master)

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

Thierry Carrez (ttx)
tags: added: havana-rc-potential
removed: havana-backport-potential
Thierry Carrez (ttx)
tags: added: havana-backport-potential
removed: havana-rc-potential
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to keystone (master)

Reviewed: https://review.openstack.org/49594
Committed: http://github.com/openstack/keystone/commit/efd46cc11873b17c83ba0b9920110147b8975364
Submitter: Jenkins
Branch: master

commit efd46cc11873b17c83ba0b9920110147b8975364
Author: David Stanek <email address hidden>
Date: Thu Oct 3 12:53:06 2013 +0000

    Treats OS-KSADM:password as password in v2 APIs

    The OS-KSADM Extension documentation uses OS-KSADM:password
    instead of password in some of its examples. This makes
    those examples work.

    Change-Id: Ibdfab0b9f085548a86eb391e7dc3b279dfc7b420
    Fixes-Bug: #1226466

Changed in keystone:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in keystone:
milestone: none → icehouse-1
status: Fix Committed → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to keystone (master)

Reviewed: https://review.openstack.org/49062
Committed: http://github.com/openstack/keystone/commit/96f1980f1f9d15594f984269e9a0d1447e1f128a
Submitter: Jenkins
Branch: master

commit 96f1980f1f9d15594f984269e9a0d1447e1f128a
Author: David Stanek <email address hidden>
Date: Mon Sep 30 21:16:25 2013 +0000

    Refactor assertEqualXML into a testtools matcher

    Not all tests inheriting from TestCase will need to check XML equality.
    Moving this functionality to a matcher leaves it up to the test class to
    decided whether or not it needs it.

    Change-Id: Ib28ec3b5dd96f662ce0cd90c650434b24c63ad6c
    Related-Bug: #1226466

Thierry Carrez (ttx)
Changed in keystone:
milestone: icehouse-1 → 2014.1
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.