smiklosovic commented on code in PR #4427:
URL: https://github.com/apache/cassandra/pull/4427#discussion_r2474281227
##########
src/java/org/apache/cassandra/auth/CassandraRoleManager.java:
##########
@@ -110,8 +98,24 @@ public class CassandraRoleManager implements IRoleManager,
CassandraRoleManagerM
public static final String DEFAULT_SUPERUSER_NAME = "cassandra";
public static final String DEFAULT_SUPERUSER_PASSWORD = "cassandra";
+ /**
+ * Role options which are supported for all authentication mechanisms.
IAuthenticator implementations can declare
+ * additional supported role options via {@link
IAuthenticator#getSupportedRoleOptions()}.
+ */
+ @VisibleForTesting
+ static final Set<Option> DEFAULT_SUPPORTED_ROLE_OPTIONS =
ImmutableSet.of(Option.LOGIN, Option.SUPERUSER);
Review Comment:
what's wrong with `Set.of(...)`, below as well? That is immutable as well.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
