search

opensearch-project / security

🔐 Secure your cluster with TLS, numerous authentication backends, data masking, audit logging as well as role-based access control on indices, documents, and fields
https://opensearch.org/docs/latest/security-plugin/index/
Apache License 2.0
180 stars 263 forks source link

[Backport 2.x] Replaced uses of SecurityRoles by Set<String> mappedRoles where the SecurityRoles functionality is not needed #4458

Closed opensearch-trigger-bot[bot] closed 2 weeks ago

opensearch-trigger-bot[bot] commented 2 weeks ago

Backport 681a94491b5a4896ed1c70467ef111efd65fa33f from #4432.

codecov[bot] commented 2 weeks ago

Codecov Report

Attention: Patch coverage is 60.00000% with 2 lines in your changes missing coverage. Please review.

Project coverage is 65.37%. Comparing base (87e98f8) to head (df0cebf).

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/opensearch-project/security/pull/4458/graphs/tree.svg?width=650&height=150&src=pr&token=rBpySfQXMt&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opensearch-project)](https://app.codecov.io/gh/opensearch-project/security/pull/4458?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opensearch-project) ```diff @@ Coverage Diff @@ ## 2.x #4458 +/- ## ========================================== - Coverage 65.38% 65.37% -0.01% ========================================== Files 311 311 Lines 21868 21863 -5 Branches 3550 3548 -2 ========================================== - Hits 14298 14294 -4 + Misses 5802 5800 -2 - Partials 1768 1769 +1 ``` | [Files](https://app.codecov.io/gh/opensearch-project/security/pull/4458?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opensearch-project) | Coverage Δ | | |---|---|---| | [...earch/security/privileges/PrivilegesEvaluator.java](https://app.codecov.io/gh/opensearch-project/security/pull/4458?src=pr&el=tree&filepath=src%2Fmain%2Fjava%2Forg%2Fopensearch%2Fsecurity%2Fprivileges%2FPrivilegesEvaluator.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opensearch-project#diff-c3JjL21haW4vamF2YS9vcmcvb3BlbnNlYXJjaC9zZWN1cml0eS9wcml2aWxlZ2VzL1ByaXZpbGVnZXNFdmFsdWF0b3IuamF2YQ==) | `72.17% <50.00%> (+0.05%)` | :arrow_up: | | [...rity/privileges/ProtectedIndexAccessEvaluator.java](https://app.codecov.io/gh/opensearch-project/security/pull/4458?src=pr&el=tree&filepath=src%2Fmain%2Fjava%2Forg%2Fopensearch%2Fsecurity%2Fprivileges%2FProtectedIndexAccessEvaluator.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opensearch-project#diff-c3JjL21haW4vamF2YS9vcmcvb3BlbnNlYXJjaC9zZWN1cml0eS9wcml2aWxlZ2VzL1Byb3RlY3RlZEluZGV4QWNjZXNzRXZhbHVhdG9yLmphdmE=) | `74.46% <66.66%> (+0.99%)` | :arrow_up: | ... and [2 files with indirect coverage changes](https://app.codecov.io/gh/opensearch-project/security/pull/4458/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opensearch-project)