yahoo-eng-team team mailing list archive

[Thierry Carrez <thierry.carrez+lp@xxxxxxxxx>]

** Changed in: keystone
       Status: Fix Committed => Fix Released

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to Keystone.
https://bugs.launchpad.net/bugs/1152283

Title:
  delete token for trust invalidation has typo

Status in OpenStack Identity (Keystone):
  Fix Released

Bug description:
  There are  a couple typos that showed delete token for trust is
  untested.

  
  In 
  keystone/identity/controllers.py

   line 171

   token_list = token_api.list_tokens(context, userid,

  userid should be user_id

  
   line 346

  delete_tokens_for_user(self.token_api, self.trust_api, context,
  				                               user_id, tenant_id)
  	
  the last argument to this function is user, not tenant_id.

  The backends can and should  ignore the user_id field if the trust_id
  is passed in to list tokens.  (The exception is the memcached backend,
  which needs the userid to find the tokens.  This is another argument
  for deprecating the memcahced backend.)

To manage notifications about this bug go to:
https://bugs.launchpad.net/keystone/+bug/1152283/+subscriptions