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 1.3] When excluding fields also exclude the term + .keyword #2385

Merged
merged 3 commits into from
Jan 12, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport cb81cc6 from #2378

…d` (#2378)

* [Backport 2.x] When excluding fields also exclude the term + `.keyword` (#2377)

(cherry picked from commit b9652fe)

* Switch to transport client for 1.x

Signed-off-by: Peter Nied <[email protected]>

Signed-off-by: Peter Nied <[email protected]>
Co-authored-by: opensearch-trigger-bot[bot] <98922864+opensearch-trigger-bot[bot]@users.noreply.github.com>
Co-authored-by: Peter Nied <[email protected]>
(cherry picked from commit cb81cc6)
@opensearch-trigger-bot opensearch-trigger-bot bot requested a review from a team January 6, 2023 20:25
@peternied peternied changed the title [Backport 1.3] [Backport 1.x] When excluding fields also exclude the term + .keyword [Backport 1.3] When excluding fields also exclude the term + .keyword Jan 6, 2023
@codecov-commenter
Copy link

codecov-commenter commented Jan 6, 2023

Codecov Report

Merging #2385 (cb81cc6) into 1.3 (43244d2) will increase coverage by 0.22%.
The diff coverage is 73.68%.

❗ Current head cb81cc6 differs from pull request most recent head 4b9619b. Consider uploading reports for the commit 4b9619b to get more accurate results

@@             Coverage Diff              @@
##                1.3    #2385      +/-   ##
============================================
+ Coverage     64.43%   64.66%   +0.22%     
- Complexity     3219     3223       +4     
============================================
  Files           247      247              
  Lines         17344    17365      +21     
  Branches       3073     3086      +13     
============================================
+ Hits          11176    11229      +53     
+ Misses         4622     4588      -34     
- Partials       1546     1548       +2     
Impacted Files Coverage Δ
...earch/security/resolver/IndexResolverReplacer.java 63.97% <54.54%> (+0.10%) ⬆️
...security/configuration/DlsFlsFilterLeafReader.java 62.44% <100.00%> (+1.02%) ⬆️
...security/dlic/rest/api/InternalUsersApiAction.java 83.78% <100.00%> (+1.43%) ⬆️
...earch/security/privileges/PrivilegesEvaluator.java 71.57% <100.00%> (ø)
...g/opensearch/security/auditlog/sink/Log4JSink.java 48.00% <0.00%> (-12.00%) ⬇️
...search/security/securityconf/impl/v7/ConfigV7.java 72.12% <0.00%> (-1.82%) ⬇️
...search/security/configuration/DlsFlsValveImpl.java 70.22% <0.00%> (-1.52%) ⬇️
...g/opensearch/security/support/WildcardMatcher.java 61.03% <0.00%> (-0.65%) ⬇️
...pensearch/security/securityconf/ConfigModelV7.java 63.90% <0.00%> (-0.57%) ⬇️
...ensearch/security/ssl/DefaultSecurityKeyStore.java 68.93% <0.00%> (-0.15%) ⬇️
... and 16 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@RyanL1997 RyanL1997 merged commit f549b93 into 1.3 Jan 12, 2023
@RyanL1997 RyanL1997 deleted the backport/backport-2378-to-1.3 branch January 12, 2023 23:53
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.

4 participants