[
https://issues.apache.org/jira/browse/CASSANDRA-21051?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18042524#comment-18042524
]
Stefan Miklosovic commented on CASSANDRA-21051:
-----------------------------------------------
I run just multiplexers since this does not touch anything else. It run 2500x
without being flaky so I am +1.
[CASSANDRA-21051|https://github.com/instaclustr/cassandra/tree/CASSANDRA-21051]
{noformat}
java17_pre-commit_tests
java17_separate_tests
✓ j17_build 10m 34s
✓ j17_unit_tests_repeat 13m 17s
✓ j17_utests_cdc_repeat 13m 18s
✓ j17_utests_latest_repeat 12m 28s
✓ j17_utests_oa_repeat 11m 23s
java11_pre-commit_tests
java11_separate_tests
✓ j11_build 10m 38s
✓ j11_unit_tests_repeat 13m 5s
{noformat}
[java17_pre-commit_tests|https://app.circleci.com/pipelines/github/instaclustr/cassandra/6138/workflows/bd685120-4d4e-4c65-bc9a-be565d43aa5c]
[java17_separate_tests|https://app.circleci.com/pipelines/github/instaclustr/cassandra/6138/workflows/2c3b9383-493e-4625-ab34-d4afe07ef5e6]
[java11_pre-commit_tests|https://app.circleci.com/pipelines/github/instaclustr/cassandra/6138/workflows/56a505cb-863a-48ea-baa7-e7385ba0661d]
[java11_separate_tests|https://app.circleci.com/pipelines/github/instaclustr/cassandra/6138/workflows/9b13be68-4d94-4573-98aa-099358d8c3c4]
> Add test for check permissions after authorize
> ----------------------------------------------
>
> Key: CASSANDRA-21051
> URL: https://issues.apache.org/jira/browse/CASSANDRA-21051
> Project: Apache Cassandra
> Issue Type: Task
> Components: Test/unit
> Reporter: Valery Baranov
> Assignee: Valery Baranov
> Priority: Normal
> Fix For: 5.x
>
> Time Spent: 0.5h
> Remaining Estimate: 0h
>
> There is no test covering this case: a user must have more than just the
> AUTHORIZE permission to grant permissions to others.
> {code:java}
> //
> src/java/org/apache/cassandra/cql3/statements/PermissionsManagementStatement.java
> public void authorize(ClientState state) throws UnauthorizedException
> {
> ...
> for (Permission p : permissions)
> state.ensurePermission(p, resource);
> }{code}
> If I remove these two lines, the existing tests will still pass.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
