Make plugins.security.dfm_empty_overrides_all dynamically toggleable#6016
Make plugins.security.dfm_empty_overrides_all dynamically toggleable#6016cwperks wants to merge 10 commits intoopensearch-project:mainfrom
plugins.security.dfm_empty_overrides_all dynamically toggleable#6016Conversation
Signed-off-by: Craig Perkins <cwperx@amazon.com>
Signed-off-by: Craig Perkins <cwperx@amazon.com>
Signed-off-by: Craig Perkins <cwperx@amazon.com>
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #6016 +/- ##
=======================================
Coverage 73.82% 73.83%
=======================================
Files 439 439
Lines 27122 27132 +10
Branches 4025 4026 +1
=======================================
+ Hits 20024 20033 +9
+ Misses 5192 5190 -2
- Partials 1906 1909 +3
🚀 New features to boost your workflow:
|
Signed-off-by: Craig Perkins <cwperx@amazon.com>
|
Generally cool for me, but I am wondering whether we should perspectively pivot towards deprecating this and just having this on by default. In the end, this is IMHO just a backwards compat setting. |
I'd be aligned with that. What I'm interested most for with this change is showing that we can authz settings updates (for sensitive settings via the |
Signed-off-by: Craig Perkins <cwperx@amazon.com>
Signed-off-by: Craig Perkins <cwperx@amazon.com>
Description
Companion core PR to make sure we authz this like any other security setting: opensearch-project/OpenSearch#20901
This PR makes
plugins.security.dfm_empty_overrides_alltoggleable at runtime by the cluster admin via Cluster settings APIEnhancement
Issues Resolved
Resolves #6002
Check List
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.