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
191 stars 273 forks source link

[Backport 2.x] Extracted the user attr handling methods from ConfigModelV7 into its own class #4431

Closed opensearch-trigger-bot[bot] closed 3 months ago

opensearch-trigger-bot[bot] commented 3 months ago

Backport 00c31e9b2b64901c0a017d0b534fc712e5ef2f1e from #4416.

codecov[bot] commented 3 months ago

Codecov Report

Attention: Patch coverage is 78.57143% with 6 lines in your changes missing coverage. Please review.

Project coverage is 65.39%. Comparing base (932bf72) to head (2d2112d). 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/4431/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/4431?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 #4431 +/- ## ========================================== + Coverage 65.36% 65.39% +0.03% ========================================== Files 309 311 +2 Lines 21839 21867 +28 Branches 3547 3550 +3 ========================================== + Hits 14275 14301 +26 + Misses 5802 5800 -2 - Partials 1762 1766 +4 ``` | [Files](https://app.codecov.io/gh/opensearch-project/security/pull/4431?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opensearch-project) | Coverage Δ | | |---|---|---| | [...pensearch/security/securityconf/ConfigModelV7.java](https://app.codecov.io/gh/opensearch-project/security/pull/4431?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.68% <100.00%> (-0.48%)` | :arrow_down: | | [...opensearch/security/privileges/UserAttributes.java](https://app.codecov.io/gh/opensearch-project/security/pull/4431?src=pr&el=tree&filepath=src%2Fmain%2Fjava%2Forg%2Fopensearch%2Fsecurity%2Fprivileges%2FUserAttributes.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opensearch-project#diff-c3JjL21haW4vamF2YS9vcmcvb3BlbnNlYXJjaC9zZWN1cml0eS9wcml2aWxlZ2VzL1VzZXJBdHRyaWJ1dGVzLmphdmE=) | `76.00% <76.00%> (ø)` | | ... and [6 files with indirect coverage changes](https://app.codecov.io/gh/opensearch-project/security/pull/4431/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opensearch-project)