[
https://issues.apache.org/jira/browse/CASSANDRA-20834?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18071714#comment-18071714
]
Andy Tolbert commented on CASSANDRA-20834:
------------------------------------------
+1, PR looks great, thank you [~jshepherd] ! I ran it through CI and trunk
appears to have a lot of flaky accord tests, i'm experiencing the same kind of
test failures on other branches. There are some known flakes here too
CASSANDRA-19542 CASSANDRA-20306 CASSANDRA-20509(which was fixed but maybe needs
revisit), but the added tests all pass and things look good otherwise.
* [^CASSANDRA-20834-Apr-6-2_ci_summary.html]
* [^CASSANDRA-20834-Apr-6-2_results_details.tar.gz]
> Remove getAuthenticator() references for determining if authn is required
> -------------------------------------------------------------------------
>
> Key: CASSANDRA-20834
> URL: https://issues.apache.org/jira/browse/CASSANDRA-20834
> Project: Apache Cassandra
> Issue Type: Improvement
> Components: Legacy/Core
> Reporter: Joel Shepherd
> Assignee: Joel Shepherd
> Priority: Normal
> Fix For: 6.x
>
> Attachments: CASSANDRA-20834-Apr-6-2_ci_summary.html,
> CASSANDRA-20834-Apr-6-2_results_details.tar.gz
>
>
> Remove calls to DatabaseDescriptor's getAuthenticator() for determining
> whether the node is enforcing authentication or not, and replace with
> isAuthenticationRequired().
> This is a prerequisite to implementing per-connection authenticator
> selection, where "authentication required" will depend on context (node- or
> session-level) and the configured authenticators available to the node.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
