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 <[email protected]> Gerrit-Reviewer: Alon Bar-Lev <[email protected]> Gerrit-Reviewer: Barak Azulay <[email protected]> Gerrit-Reviewer: Juan Hernandez <[email protected]> Gerrit-Reviewer: Yair Zaslavsky <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
