Yair Zaslavsky has posted comments on this change. Change subject: aaa: Introducing format to id of User and Group API entities ......................................................................
Patch Set 5: Hi Juan, aobut your comment - "I have to insist that it isn't acceptable to have two possible identifiers (and thus two possible URIs) for the same resource, as we can do with just one. Having more than one possible identifier just adds confusion. As we can easily do everything just with the external identifier we should use it everywhere. Thus the "id=" and "providerid=" prefixes aren't needed." Anywhere means not use "user", "group", but also "permission", right? We are in favor of this approach. This means that the primary key for users (AKA internal id) will be hidden and not presented anywhere in the API. I would like to expose besides the "id" (hex of provider:providerid) also the "provider" and "providerid" as two new attributes for user/group -- To view, visit http://gerrit.ovirt.org/26191 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Iaecb5d43945769db82475edde1c7075c1a343c07 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yair Zaslavsky <yzasl...@redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com> Gerrit-Reviewer: Barak Azulay <bazu...@redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernan...@redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
