yahoo-eng-team team mailing list archive

Thread
Date

[Bug 1214135] Re: add foreign keys to oauth tables

To: yahoo-eng-team@xxxxxxxxxxxxxxxxxxx
From: Thierry Carrez <thierry.carrez+lp@xxxxxxxxx>
Date: Thu, 05 Sep 2013 09:26:39 -0000
Reply-to: Bug 1214135 <1214135@xxxxxxxxxxxxxxxxxx>
Sender: bounces@xxxxxxxxxxxxx

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

** Changed in: keystone
    Milestone: None => havana-3

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

Title:
  add foreign keys to oauth tables

Status in OpenStack Identity (Keystone):
  Fix Released

Bug description:
  As suggested by Adam, the consumer_id should be a foreign key in the RequestToken and AccessToken tables, since it exists in the ConsumerTable.
  https://review.openstack.org/#/c/29130/64/keystone/contrib/oauth1/migrate_repo/versions/001_add_oauth_tables.py

  If these changes were to be implemented, then 'delete_consumer' would also need a slight change too.
  https://review.openstack.org/#/c/29130/64/keystone/contrib/oauth1/backends/sql.py

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