Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Backport 2.x] [Enhancement-3191] transport_enabled setting on an auth domain and authorizer may be unnecessary after transport client removal #3966

Merged
merged 1 commit into from
Jan 22, 2024

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 881ed58 from #3939.

…authorizer may be unnecessary after transport client removal (#3939)

Signed-off-by: Prabhas Kurapati <[email protected]>
(cherry picked from commit 881ed58)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Copy link

codecov bot commented Jan 19, 2024

Codecov Report

Attention: 8 lines in your changes are missing coverage. Please review.

Comparison is base (a0c3dba) 65.39% compared to head (6e4314e) 65.46%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##              2.x    #3966      +/-   ##
==========================================
+ Coverage   65.39%   65.46%   +0.06%     
==========================================
  Files         297      297              
  Lines       21051    21051              
  Branches     3447     3443       -4     
==========================================
+ Hits        13767    13780      +13     
+ Misses       5574     5562      -12     
+ Partials     1710     1709       -1     
Files Coverage Δ
...ch/security/securityconf/DynamicConfigModelV7.java 65.51% <100.00%> (-0.79%) ⬇️
...pensearch/security/setting/DeprecatedSettings.java 85.71% <100.00%> (+5.71%) ⬆️
...ch/security/securityconf/DynamicConfigModelV6.java 0.00% <0.00%> (ø)
...search/security/securityconf/impl/v7/ConfigV7.java 78.39% <83.33%> (+1.42%) ⬆️
...search/security/securityconf/impl/v6/ConfigV6.java 62.39% <66.66%> (+2.58%) ⬆️

... and 1 file with indirect coverage changes

@peternied peternied merged commit e008610 into 2.x Jan 22, 2024
88 checks passed
@peternied peternied deleted the backport/backport-3939-to-2.x branch January 22, 2024 14:24
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