From 706612e300ffc039ee35c07601cb60dab0486945 Mon Sep 17 00:00:00 2001 From: Aleksey Yeschenko Date: Mon, 17 Dec 2012 18:35:31 +0300 Subject: [PATCH] Fix SimpleAuthorizer example; patch by Aleksey Yeschenko, reviewed by Jonathan Ellis for CASSANDRA-5072 --- CHANGES.txt | 1 + .../org/apache/cassandra/auth/SimpleAuthorizer.java | 10 +++++----- 2 files changed, 6 insertions(+), 5 deletions(-) diff --git a/CHANGES.txt b/CHANGES.txt index 327d42795c..520cf08440 100644 --- a/CHANGES.txt +++ b/CHANGES.txt @@ -5,6 +5,7 @@ Merged from 1.1: * fix temporarily missing schema after upgrade from pre-1.1.5 (CASSANDRA-5061) * Fix ALTER TABLE overriding compression options with defaults (CASSANDRA-4996, CASSANDRA-5066) + * Fix SimpleAuthorizer example (CASSANDRA-5072) 1.2-rc1 diff --git a/examples/simple_authentication/src/org/apache/cassandra/auth/SimpleAuthorizer.java b/examples/simple_authentication/src/org/apache/cassandra/auth/SimpleAuthorizer.java index 2d7b6442de..df626547a6 100644 --- a/examples/simple_authentication/src/org/apache/cassandra/auth/SimpleAuthorizer.java +++ b/examples/simple_authentication/src/org/apache/cassandra/auth/SimpleAuthorizer.java @@ -41,10 +41,10 @@ public class SimpleAuthorizer extends LegacyAuthorizer public EnumSet authorize(AuthenticatedUser user, List resource) { if (resource.size() < 2 || !Resources.ROOT.equals(resource.get(0)) || !Resources.KEYSPACES.equals(resource.get(1))) - return EnumSet.copyOf(Permission.NONE); + return EnumSet.noneOf(Permission.class); String keyspace, columnFamily = null; - EnumSet authorized = EnumSet.copyOf(Permission.NONE); + EnumSet authorized = EnumSet.noneOf(Permission.class); // /cassandra/keyspaces if (resource.size() == 2) @@ -82,7 +82,7 @@ public class SimpleAuthorizer extends LegacyAuthorizer { String kspAdmins = accessProperties.getProperty(KEYSPACES_WRITE_PROPERTY); for (String admin : kspAdmins.split(",")) - if (admin.equals(user.username)) + if (admin.equals(user.getName())) return EnumSet.copyOf(Permission.ALL); } @@ -104,7 +104,7 @@ public class SimpleAuthorizer extends LegacyAuthorizer { for (String reader : readers.split(",")) { - if (reader.equals(user.username)) + if (reader.equals(user.getName())) { canRead = true; break; @@ -116,7 +116,7 @@ public class SimpleAuthorizer extends LegacyAuthorizer { for (String writer : writers.split(",")) { - if (writer.equals(user.username)) + if (writer.equals(user.getName())) { canWrite = true; break;