Skip to content

Rename authenticated user transport variables - #6432

Merged
cwperks merged 1 commit into
opensearch-project:mainfrom
cwperks:agent/rename-authenticated-user-variables
Aug 25, 2026
Merged

Rename authenticated user transport variables#6432
cwperks merged 1 commit into
opensearch-project:mainfrom
cwperks:agent/rename-authenticated-user-variables

Conversation

@cwperks

@cwperks cwperks commented Aug 25, 2026

Copy link
Copy Markdown
Member

Follow-up to #6419. Removes legacy Subject-oriented local variable names left behind after authenticated users began being stored directly as User instances.

This renames transport-local variables and updates comments in SecurityInterceptor and SecurityRequestHandler. Existing ThreadContext keys and transport header constants remain unchanged for compatibility.

Validation

  • ./gradlew spotlessApply
  • ./gradlew test --tests org.opensearch.security.transport.SecurityInterceptorTests --tests org.opensearch.security.transport.RestoringTransportResponseHandlerTest
  • git diff --check

Signed-off-by: Craig Perkins <craig5008@gmail.com>
@github-actions

Copy link
Copy Markdown
Contributor

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

🧪 No relevant tests
🔒 No security concerns identified
✅ No TODO sections
🔀 No multiple PR themes
⚡ No major issues detected

@codecov

codecov Bot commented Aug 25, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 88.23529% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 75.41%. Comparing base (d1b600b) to head (a141a2f).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
...search/security/transport/SecurityInterceptor.java 87.50% 0 Missing and 1 partial ⚠️
...rch/security/transport/SecurityRequestHandler.java 88.88% 0 Missing and 1 partial ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #6432      +/-   ##
==========================================
- Coverage   75.51%   75.41%   -0.10%     
==========================================
  Files         456      456              
  Lines       30282    30282              
  Branches     4574     4574              
==========================================
- Hits        22866    22838      -28     
- Misses       5284     5311      +27     
- Partials     2132     2133       +1     
Files with missing lines Coverage Δ
...search/security/transport/SecurityInterceptor.java 79.34% <87.50%> (ø)
...rch/security/transport/SecurityRequestHandler.java 59.28% <88.88%> (ø)

... and 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.

@cwperks
cwperks merged commit 55915be into opensearch-project:main Aug 25, 2026
109 of 114 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants