Alon Bar-Lev has posted comments on this change. Change subject: aaa: sync: handle group loops ......................................................................
Patch Set 1: (1 comment) http://gerrit.ovirt.org/#/c/35663/1/backend/manager/modules/aaa/src/main/java/org/ovirt/engine/core/aaa/AuthzUtils.java File backend/manager/modules/aaa/src/main/java/org/ovirt/engine/core/aaa/AuthzUtils.java: Line 136: groupsCache, Line 137: loopPrevention Line 138: ) Line 139: ); Line 140: loopPrevention.pop(); > Although it's obvious the rational for the data structure came from your OS but then we won't be able to log the stack when error per the order to understand what happened. Line 141: } Line 142: } Line 143: entity.put(key, groups); Line 144: return entity; -- To view, visit http://gerrit.ovirt.org/35663 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ie90ad3cbef9675be8c0ceba47c22609a9985b518 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <[email protected]> Gerrit-Reviewer: Alon Bar-Lev <[email protected]> Gerrit-Reviewer: Oved Ourfali <[email protected]> Gerrit-Reviewer: Yair Zaslavsky <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
