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] Check block request only if system index #4469

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

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

Backport 1d1b1ed7bb0d0c99e693c3571231d1f2b8b7cb7c from #4430.

codecov[bot] commented 1 week ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 65.37%. Comparing base (b3b63dc) to head (8c5f090). Report is 1 commits behind head on 2.x.

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/opensearch-project/security/pull/4469/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/4469?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 #4469 +/- ## ========================================== + Coverage 65.35% 65.37% +0.01% ========================================== Files 311 311 Lines 21863 21866 +3 Branches 3548 3550 +2 ========================================== + Hits 14289 14294 +5 Misses 5806 5806 + Partials 1768 1766 -2 ``` | [Files](https://app.codecov.io/gh/opensearch-project/security/pull/4469?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opensearch-project) | Coverage Δ | | |---|---|---| | [...ty/configuration/SecurityIndexSearcherWrapper.java](https://app.codecov.io/gh/opensearch-project/security/pull/4469?src=pr&el=tree&filepath=src%2Fmain%2Fjava%2Forg%2Fopensearch%2Fsecurity%2Fconfiguration%2FSecurityIndexSearcherWrapper.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opensearch-project#diff-c3JjL21haW4vamF2YS9vcmcvb3BlbnNlYXJjaC9zZWN1cml0eS9jb25maWd1cmF0aW9uL1NlY3VyaXR5SW5kZXhTZWFyY2hlcldyYXBwZXIuamF2YQ==) | `94.73% <100.00%> (+0.29%)` | :arrow_up: | ... and [2 files with indirect coverage changes](https://app.codecov.io/gh/opensearch-project/security/pull/4469/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opensearch-project)