belongsTo should validate tenant name not id

[Michael Basnight]

the spec is very vague on this, but token validation in openstack common and nova use the tenant name as the "project". Also rackspace uses the tenant name currently in production for belongsTo. I propose to check the name instead of the id for these reasons.

[OpenStack Infra (hudson-openstack)]

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

[Joseph Heck (heckj)]

seems like a bad idea going forward to match belongsTo (on behalf of... tokens) to a name and not the ID.

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/5062
Committed: http://github.com/openstack/keystone/commit/5c6bccf1c57b06a19845c696c19274ae9f080104
Submitter: Jenkins
Branch: master

commit 5c6bccf1c57b06a19845c696c19274ae9f080104
Author: Michael Basnight <email address hidden>
Date: Wed Mar 7 22:32:23 2012 -0600

    fixes lp#949648 change belongsTo validate to name

    Change-Id: I3d36290ad95a0440c006e2daff5b831be62957ae

[OpenStack Infra (hudson-openstack)]

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

[Dolph Mathews (dolph)]

Reverted in https://review.openstack.org/#change,5564

[Mark Miller (mark-m-miller)]

I am wondering why you don't simply add provisions for both the tenant ID and the tenant name. Change "belongsTo" to "belongsToId" and add "belongsToName"?

[Michael Basnight (hubcap)]

Well thats a question for the next version of the spec. This was already in the existing spec and had to be fixed. :D