<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Apr 12, 2017 at 2:47 PM, David Stanek <span dir="ltr"><<a href="mailto:dstanek@dstanek.com" target="_blank">dstanek@dstanek.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 12-Apr 14:30, Rodrigo Duarte wrote:<br>
> Just to illustrate the discussion, we have a bug fix that currently tries<br>
> to drop a FK between the federation and identity subsystems [1].<br>
><br>
</span>> [1] <a href="https://review.openstack.org/#/c/445505/" rel="noreferrer" target="_blank">https://review.openstack.org/#<wbr>/c/445505/</a><br>
<br>
I think this highlights one of my problems with the current architecture. I see that<br>
you've removed the FK and added delete logic to do what the data layer would be doing<br>
for you. I didn't see any added get_user() checks to make sure the user_id being used<br>
in creates/updates is valid. Are we already checking that somewhere else or is this<br>
introducing a new bug?<br></blockquote><div><br></div><div>The review [1] is dropping the idp_id and idp_id + protocol_id FKs, not the user_id one.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5"><br>
--<br>
david stanek<br>
web: <a href="https://dstanek.com" rel="noreferrer" target="_blank">https://dstanek.com</a><br>
twitter: <a href="https://twitter.com/dstanek" rel="noreferrer" target="_blank">https://twitter.com/dstanek</a><br>
<br>
______________________________<wbr>______________________________<wbr>______________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.<wbr>openstack.org?subject:<wbr>unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/<wbr>cgi-bin/mailman/listinfo/<wbr>openstack-dev</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><font color="#666666">Rodrigo Duarte Sousa<br></font></div><div><font color="#666666">Senior Quality Engineer @ Red Hat<br></font></div><div dir="ltr"><div><div><span style="color:rgb(102,102,102)">MSc</span><span style="color:rgb(102,102,102)"></span><span style="color:rgb(102,102,102)"> in Computer Science</span><br><font color="#3333ff"><a href="http://rodrigods.com" target="_blank">http://<font color="#3333ff">rodrigods.com</font></a></font></div></div></div></div></div></div></div></div></div></div></div></div>
</div></div>