yahoo-eng-team team mailing list archive

[Alexey Miroshkin <alexey.miroshkin@xxxxxxxxxx>]

Public bug reported:

The current implementation of credential.backends.sql.Credential work with filters directly:  list_credentials def list_credentials(self, **filters):
..
if 'user_id' in filters:
        query = query.filter_by(user_id=filters.get('user_id'))

The most part of backends implementation follows hints model,
considering any filters as hints, so the particular backend
implementation has an option to process it or ignore it. It makes sense
to update credential backend,  for example:

def list_credentials(self, hints)

** Affects: keystone
     Importance: Undecided
     Assignee: Alexey Miroshkin (amirosh)
         Status: New

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

Title:
  Credential backend doesn't use hints model

Status in OpenStack Identity (Keystone):
  New

Bug description:
  The current implementation of credential.backends.sql.Credential work with filters directly:  list_credentials def list_credentials(self, **filters):
  ..
  if 'user_id' in filters:
          query = query.filter_by(user_id=filters.get('user_id'))

  The most part of backends implementation follows hints model,
  considering any filters as hints, so the particular backend
  implementation has an option to process it or ignore it. It makes
  sense to update credential backend,  for example:

  def list_credentials(self, hints)

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