On Wed, Apr 12, 2017 at 2:47 PM, David Stanek <dstanek at dstanek.com> wrote: > On 12-Apr 14:30, Rodrigo Duarte wrote: > > Just to illustrate the discussion, we have a bug fix that currently tries > > to drop a FK between the federation and identity subsystems [1]. > > > > [1] https://review.openstack.org/#/c/445505/ > > I think this highlights one of my problems with the current architecture. > I see that > you've removed the FK and added delete logic to do what the data layer > would be doing > for you. I didn't see any added get_user() checks to make sure the user_id > being used > in creates/updates is valid. Are we already checking that somewhere else > or is this > introducing a new bug? > The review [1] is dropping the idp_id and idp_id + protocol_id FKs, not the user_id one. > > -- > david stanek > web: https://dstanek.com > twitter: https://twitter.com/dstanek > > __________________________________________________________________________ > OpenStack Development Mailing List (not for usage questions) > Unsubscribe: OpenStack-dev-request at lists.openstack.org?subject:unsubscribe > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev > -- Rodrigo Duarte Sousa Senior Quality Engineer @ Red Hat MSc in Computer Science http://rodrigods.com -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20170412/649b4c2b/attachment.html>