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] New algorithm for resolving action groups #4472

Closed opensearch-trigger-bot[bot] closed 1 week ago

opensearch-trigger-bot[bot] commented 1 week ago

Backport 20291a950d7a975db13ac062cddf61120ba2a8e0 from #4448.

codecov[bot] commented 1 week ago

Codecov Report

Attention: Patch coverage is 82.75862% with 10 lines in your changes missing coverage. Please review.

Project coverage is 65.43%. Comparing base (5984477) to head (34cc23e).

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/opensearch-project/security/pull/4472/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/4472?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 #4472 +/- ## ========================================== + Coverage 65.37% 65.43% +0.06% ========================================== Files 311 312 +1 Lines 21866 21888 +22 Branches 3550 3553 +3 ========================================== + Hits 14294 14322 +28 + Misses 5804 5799 -5 + Partials 1768 1767 -1 ``` | [Files](https://app.codecov.io/gh/opensearch-project/security/pull/4472?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opensearch-project) | Coverage Δ | | |---|---|---| | [...ecurityconf/impl/SecurityDynamicConfiguration.java](https://app.codecov.io/gh/opensearch-project/security/pull/4472?src=pr&el=tree&filepath=src%2Fmain%2Fjava%2Forg%2Fopensearch%2Fsecurity%2Fsecurityconf%2Fimpl%2FSecurityDynamicConfiguration.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opensearch-project#diff-c3JjL21haW4vamF2YS9vcmcvb3BlbnNlYXJjaC9zZWN1cml0eS9zZWN1cml0eWNvbmYvaW1wbC9TZWN1cml0eUR5bmFtaWNDb25maWd1cmF0aW9uLmphdmE=) | `80.30% <100.00%> (+0.77%)` | :arrow_up: | | [...pensearch/security/securityconf/ConfigModelV7.java](https://app.codecov.io/gh/opensearch-project/security/pull/4472?src=pr&el=tree&filepath=src%2Fmain%2Fjava%2Forg%2Fopensearch%2Fsecurity%2Fsecurityconf%2FConfigModelV7.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opensearch-project#diff-c3JjL21haW4vamF2YS9vcmcvb3BlbnNlYXJjaC9zZWN1cml0eS9zZWN1cml0eWNvbmYvQ29uZmlnTW9kZWxWNy5qYXZh) | `67.30% <80.00%> (-0.21%)` | :arrow_down: | | [...h/security/securityconf/FlattenedActionGroups.java](https://app.codecov.io/gh/opensearch-project/security/pull/4472?src=pr&el=tree&filepath=src%2Fmain%2Fjava%2Forg%2Fopensearch%2Fsecurity%2Fsecurityconf%2FFlattenedActionGroups.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opensearch-project#diff-c3JjL21haW4vamF2YS9vcmcvb3BlbnNlYXJjaC9zZWN1cml0eS9zZWN1cml0eWNvbmYvRmxhdHRlbmVkQWN0aW9uR3JvdXBzLmphdmE=) | `81.25% <81.25%> (ø)` | | ... and [5 files with indirect coverage changes](https://app.codecov.io/gh/opensearch-project/security/pull/4472/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opensearch-project)