Skip to content

Add Finnegan Carroll (gh: finnegancarroll) as a co-maintainer of the Security repo#6031

Merged
cwperks merged 1 commit intoopensearch-project:mainfrom
cwperks:finn
Mar 24, 2026
Merged

Add Finnegan Carroll (gh: finnegancarroll) as a co-maintainer of the Security repo#6031
cwperks merged 1 commit intoopensearch-project:mainfrom
cwperks:finn

Conversation

@cwperks
Copy link
Member

@cwperks cwperks commented Mar 23, 2026

Description

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

Finnegan has made strong and sustained contributions to the repo, especially in the gRPC / auxiliary transport area, and has helped move several important security capabilities from design through implementation. In addition to code contributions, he has also opened and driven multiple well-scoped feature and meta issues that have helped shape the roadmap for this area.

Some notable code contributions include:

  • JWT authentication for gRPC transport: #5916
  • [Bugfix] Make gRPC JWT header keys case insensitive: #5929
  • TLS support for auxiliary transports: #5375
  • Bump gradle to 8.14.3 and move gRPC transport out of experimental: #5542
  • Bump test only protobuf to 0.13.0: #5553
    Format SSLConfigConstants.java and fix typos: #5145

Finnegan has also contributed valuable design and roadmap work through issues and meta tracking items, including:

  • [Meta] Authn/Authz support for gRPC/Auxiliary transports in security plugin: #5813
  • [FEATURE] JWT support for auxiliary transports (gRPC): #5812
  • [FEATURE] Provide gRPC interceptor alternative to RestHandler: #5785
  • [FEATURE] Enable hot reload of gRPC certificates: #5531
  • [META] Testing strategy for auxiliary transports: #5524
  • [FEATURE] Migrate all SecureTransportSettingsProviders to SSLContexts: #5258
    [FEATURE] Provide SSL configuration for auxiliary transports: #5104

His contribution history shows both breadth and depth: he has authored merged PRs across security transport, TLS, dependency maintenance, and release-related work, and his recent work has been especially important for enabling security support around auxiliary transports and gRPC.

Check List

  • New functionality includes testing
  • New functionality has been documented
  • New Roles/Permissions have a corresponding security dashboards plugin PR
  • API changes companion pull request created
  • Commits are signed per the DCO using --signoff

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.

…Security repo

Signed-off-by: Craig Perkins <cwperx@amazon.com>
@cwperks
Copy link
Member Author

cwperks commented Mar 23, 2026

Github Request raised: opensearch-project/.github#478

@codecov
Copy link

codecov bot commented Mar 24, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.86%. Comparing base (2b86187) to head (37d3c6d).
⚠️ Report is 7 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #6031      +/-   ##
==========================================
- Coverage   73.88%   73.86%   -0.02%     
==========================================
  Files         440      440              
  Lines       27229    27229              
  Branches     4044     4044              
==========================================
- Hits        20119    20114       -5     
- Misses       5199     5205       +6     
+ Partials     1911     1910       -1     

see 9 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.

@reta
Copy link
Collaborator

reta commented Mar 24, 2026

Welcome @finnegancarroll !!!

@cwperks cwperks merged commit bbb467c into opensearch-project:main Mar 24, 2026
123 of 127 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants