> On June 1, 2017, 9:47 p.m., Patrick Rhomberg wrote: > > geode-core/src/main/java/org/apache/geode/management/internal/security/ResourceOperation.java > > Line 29 (original), 30 (patched) > > <https://reviews.apache.org/r/59692/diff/4/?file=1738269#file1738269line33> > > > > Is it possible to make this `@Repeatable`? There are some operations > > that require multiple security permissions, and it would be nice to be able > > to just annotate those functions twice. > > > > For instance, `DistributedSystemMXBean.backupAllMembers` should have > > `DATA:READ` and `CLUSTER:WRITE:DISK`.
I tried to do this, but it's more involved than just adding the repeatable annoation here. The user of these annotations will need to be updated to handle multiple values. Possibly for future enhancement. - Jinmei ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59692/#review176684 ----------------------------------------------------------- On June 2, 2017, 2:31 p.m., Jinmei Liao wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/59692/ > ----------------------------------------------------------- > > (Updated June 2, 2017, 2:31 p.m.) > > > Review request for geode, Emily Yeh, Jared Stewart, Ken Howe, Kirk Lund, and > Patrick Rhomberg. > > > Repository: geode > > > Description > ------- > > GEODE-2925: add target for resource operation for finer grained security > > > Diffs > ----- > > > geode-core/src/main/java/org/apache/geode/examples/security/ExampleSecurityManager.java > 84f97de56 > > geode-core/src/main/java/org/apache/geode/internal/security/IntegratedSecurityService.java > f9fade1cf > > geode-core/src/main/java/org/apache/geode/internal/security/SecurityService.java > 14784c391 > > geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/DiskStoreCommands.java > 64fafda84 > > geode-core/src/main/java/org/apache/geode/management/internal/security/AccessControlMBean.java > 6514a33e5 > > geode-core/src/main/java/org/apache/geode/management/internal/security/MBeanServerWrapper.java > fe79efbed > > geode-core/src/main/java/org/apache/geode/management/internal/security/ResourceOperation.java > db3a1872a > geode-core/src/main/java/org/apache/geode/security/ResourcePermission.java > 45da46441 > > geode-core/src/test/java/org/apache/geode/management/internal/security/ResourcePermissionTest.java > b728b271e > > geode-core/src/test/java/org/apache/geode/management/internal/security/TestCommand.java > 3f8f4d9d4 > > geode-core/src/test/java/org/apache/geode/security/SimpleSecurityManagerTest.java > 2d6fbcaeb > geode-core/src/test/java/org/apache/geode/security/TestSecurityManager.java > 6080b5de8 > > geode-core/src/test/resources/org/apache/geode/codeAnalysis/excludedClasses.txt > 9cff80d19 > > geode-web-api/src/main/java/org/apache/geode/rest/internal/web/security/RestSecurityService.java > 80ff719b0 > > > Diff: https://reviews.apache.org/r/59692/diff/5/ > > > Testing > ------- > > precheckin runing > > > Thanks, > > Jinmei Liao > >