<div dir="ltr">Hi Tristan,<div><br></div><div>This is expected, but is also a bug -- this endpoint currently saves the document, so simultaneous calls to the same group document could clobber one another. The solution to this would be to use atomic update calls in the backend implementation to update the access list. I've opened a ticket for this: <a href="https://github.com/girder/girder/issues/1817">https://github.com/girder/girder/issues/1817</a></div><div><br></div><div>Thanks for reporting this.</div><div><br></div><div><br></div></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr">Zach Mullen<br>Kitware, Inc.<br>919-869-8858</div></div></div></div></div>
<br><div class="gmail_quote">On Tue, Feb 28, 2017 at 12:51 PM, Tristan Wright <span dir="ltr"><<a href="mailto:tristan.wright@kitware.com" target="_blank">tristan.wright@kitware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Followup: I <i>am </i>able to reproduce with the curl commands with the attached script. The script uses env variables GIRDER_TOKEN, USER1, USER2 (user Id's), and GROUP_ID.</div><div class="gmail_extra"><br><div class="gmail_quote"><span class="">On Tue, Feb 28, 2017 at 9:32 AM, Tristan Wright <span dir="ltr"><<a href="mailto:tristan.wright@kitware.com" target="_blank">tristan.wright@kitware.com</a>></span> wrote:<br></span><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><span class=""><div style="font-size:12.8px"><div>Hi all, </div><div>I believe I'm running into a concurrency issue with adding multiple users to a group with the "force=true" option on.</div></div><div style="font-size:12.8px"><br></div><div style="font-size:12.8px">Video of network log: <a href="https://drive.google.com/open?id=0BxhQv23DV8jmS3BxRHlTUFBscU0" target="_blank">https://drive.google.com/<wbr>open?id=0BxhQv23DV8jmS3BxRHlTU<wbr>FBscU0</a></div><div style="font-size:12.8px">Video summary:</div><div style="font-size:12.8px"><ul><li style="margin-left:15px">Page loads, fetches group user list, there is 1 user.<br></li><li style="margin-left:15px">Make two requests adding two users Sarah and Hannah</li><li style="margin-left:15px">group/access response is a list of two users, the original user and Hannah</li></ul><div>I expect there to be 3 users in the group/access response. Refreshing the page has no effect, it looks like something is colliding and not being changed properly?</div></div><div style="font-size:12.8px"><br></div><div style="font-size:12.8px">The call is made with chained promises. Perhaps 1/5 times the changes are correctly made to the access list. I've tried to replicate the problem with curl calls unsuccessfully, the final call has the right amount of users. I see the same behavior when trying to remove multiple users from a group. <br></div><div style="font-size:12.8px"><br></div><div style="font-size:12.8px">Is this a girder bug or am I making the invitation requests improperly?</div></span><span class="m_8190719944148206370HOEnZb"><font color="#888888"><div><br></div><span class="">-- <br><div class="m_8190719944148206370m_-922555234093178987gmail_signature"><div dir="ltr">- Tristan Wright<div><br></div><div><i>R&D Engineer</i></div><div><i>Kitware Inc.</i></div></div></div>
</span></font></span></div>
</blockquote></div><span class=""><br><br clear="all"><div><br></div>-- <br><div class="m_8190719944148206370gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">- Tristan Wright<div><br></div><div><i>R&D Engineer</i></div><div><i>Kitware Inc.</i></div></div></div>
</span></div>
<br>______________________________<wbr>_________________<br>
Girder-users mailing list<br>
<a href="mailto:Girder-users@public.kitware.com">Girder-users@public.kitware.<wbr>com</a><br>
<a href="http://public.kitware.com/mailman/listinfo/girder-users" rel="noreferrer" target="_blank">http://public.kitware.com/<wbr>mailman/listinfo/girder-users</a><br>
<br></blockquote></div><br></div>