← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1291705] [NEW] Oauth manager created multiple times

 

Public bug reported:

The oauth1.Manager class, which is @dependency.provider('oauth_api'), is
created multiple times when it should only be created once. A 'provider'
should only be created once because otherwise the new instance replaces
the one that's stored in the dependency map. Luckily, the oauth1.Manager
doesn't store any state so it's safe to do in this case, but it makes
for a bad example that others are copying and it might not be safe in
those cases.

** Affects: keystone
     Importance: Undecided
     Assignee: Brant Knudson (blk-u)
         Status: In Progress

** Changed in: keystone
     Assignee: (unassigned) => Brant Knudson (blk-u)

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

Title:
  Oauth manager created multiple times

Status in OpenStack Identity (Keystone):
  In Progress

Bug description:
  The oauth1.Manager class, which is @dependency.provider('oauth_api'),
  is created multiple times when it should only be created once. A
  'provider' should only be created once because otherwise the new
  instance replaces the one that's stored in the dependency map.
  Luckily, the oauth1.Manager doesn't store any state so it's safe to do
  in this case, but it makes for a bad example that others are copying
  and it might not be safe in those cases.

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


Follow ups

References