Add Shikhar Jain (GH: shikharj05) as a maintainer

[cwperks]

I have nominated and maintainers have agreed to add @shikharj05 as a co-maintainer of the security repo.

@shikharj05 has made many valuable contributions and given insightful comments on reviews and GH issues over the last few years.

Shikhar is an avid PR reviewer and has reviewed a total of 127 PRs (including open and closed) - a great quality of a maintainer :).

See all PRs where Shikhar has engaged as a commenter: https://github.com/opensearch-project/security/pulls?q=is%3Apr+commenter%3Ashikharj05

Some notable PR reviews include:

• Replace BouncyCastle's OpenBSDBCrypt use with password4j for password hashing and verification: Replace BouncyCastle's OpenBSDBCrypt use with password4j for password hashing and verification #4381
• Add support for ipv6 ip address in user injection: Add support for ipv6 ip address in user injection #4399
• Add initial changes to expose an endpoint for auth failure listener get call: Add initial changes to expose an endpoint for auth failure listener get call #4641
• Refactor SSL Configuration: Refactor SSL Configuration #4671

Shikhar also routinely opens up issues on the Security repo with valuable feature requests and reports of deficiencies (8 in total):

• See issues submitted by Shikhar here: https://github.com/opensearch-project/security/issues?q=is%3Aissue+author%3Ashikharj05

Shikhar's first issue opened was in May 2022. Notable issue submissions include:

• Details about the race condition on bootstrap that could lead to partial cache initialization: [BUG] TransportConfigUpdateAction causing a race condition during node bootstrap leading to 403/500 errors #3204
• Feature request on IP-based rate limiter to allow additional config to get original IP of request if proxied: [FEATURE/BUG] IP rate-limiting should support limiting based on remote client headers (e.g. header like X-Forwarded-For) #4262
• Audit log fails to log the request if the request is invalid: [BUG] Audit logging fails to log the request if Rest request is invalid #1849

Shikhar regularly contributes to conversation on Github issues (including RFCs):

• Insight on the RFC around Resource Sharing + Authorization: Resource Permissions and Sharing #4500 (comment)
• Engaged on the RFC around scheduled job security model: [RFC] Centralized Scheduled Job Identity Management #2905 (comment)
• Insight into ser/de issue post 2.11: [BUG] "java.lang.IllegalArgumentException: -84 is not a valid id" #4521 (comment)
• Help a community member understand security config options around dashboards read only role: [BUG] kibana_read_only role has no permissions #4626 (comment)

See all issues where Shikhar has engaged as a commenter: https://github.com/opensearch-project/security/issues?q=is%3Aissue+commenter%3Ashikharj05

Shikhar has 12 contributions on main and has additional changes open for review as well. Here are some highlights:

• Allow same certificate during hot reload: Allow same certificate during hot reload #1798
• Refactor SafeSerializationUtils for better performance: Refactor SafeSerializationUtils for better performance #4973
• Deprecate opendistro API paths: Deprecate opendistro API paths #5102
• Remove whitelist settings in favor of allowlist: Remove whitelist settings in favor of allowlist #5224
• Escape pipe character for injected user: Escape pipe character for injected users #5175
• Improve coverage of ConfigurationRepository class: Improve coverage: Adding tests for ConfigurationRepository class #5206

See all code contributions (Open + Closed) from Shikhar here: https://github.com/opensearch-project/security/commits?author=shikharj05

Additionally, Shikhar is active in the community and has given a talk at OpenSearch Con SF on attribute-based access control (ABAC):https://www.youtube.com/watch?v=zB4Mnue8mi0

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 71.69%. Comparing base (92497e2) to head (ac4df67).
Report is 3 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #5252      +/-   ##
==========================================
- Coverage   71.72%   71.69%   -0.03%     
==========================================
  Files         335      335              
  Lines       22742    22742              
  Branches     3597     3597              
==========================================
- Hits        16312    16306       -6     
- Misses       4638     4644       +6     
  Partials     1792     1792              

see 6 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.