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

Merged
merged 2 commits into from
Jan 6, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 1ec875c from #2375

Signed-off-by: Peter Nied <[email protected]>
(cherry picked from commit 1ec875c)
@opensearch-trigger-bot opensearch-trigger-bot bot requested a review from a team January 5, 2023 20:34
@codecov-commenter
Copy link

codecov-commenter commented Jan 5, 2023

Codecov Report

Merging #2377 (2af449f) into 2.x (dff0c1d) will increase coverage by 0.12%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##                2.x    #2377      +/-   ##
============================================
+ Coverage     61.05%   61.17%   +0.12%     
- Complexity     3250     3259       +9     
============================================
  Files           258      258              
  Lines         18131    18132       +1     
  Branches       3232     3232              
============================================
+ Hits          11070    11093      +23     
+ Misses         5489     5469      -20     
+ Partials       1572     1570       -2     
Impacted Files Coverage Δ
...security/configuration/DlsFlsFilterLeafReader.java 62.31% <100.00%> (+0.33%) ⬆️
...iance/ComplianceIndexingOperationListenerImpl.java 61.76% <0.00%> (-1.48%) ⬇️
...a/org/opensearch/security/tools/SecurityAdmin.java 36.00% <0.00%> (+0.24%) ⬆️
...ecurity/configuration/ConfigurationRepository.java 74.31% <0.00%> (+2.18%) ⬆️
...security/auditlog/sink/ExternalOpenSearchSink.java 61.72% <0.00%> (+2.46%) ⬆️
...ic/auth/http/jwt/AbstractHTTPJwtAuthenticator.java 59.30% <0.00%> (+3.48%) ⬆️
.../auth/http/jwt/keybyoidc/SelfRefreshingKeySet.java 63.38% <0.00%> (+3.52%) ⬆️
...nsearch/security/dlic/rest/api/AuditApiAction.java 68.08% <0.00%> (+4.25%) ⬆️
...t/keybyoidc/AuthenticatorUnavailableException.java 20.00% <0.00%> (+20.00%) ⬆️
... and 1 more

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

@peternied peternied self-assigned this Jan 5, 2023
@peternied peternied added the backport 1.x backport to 1.x branch label Jan 5, 2023
@peternied peternied merged commit b9652fe into 2.x Jan 6, 2023
@peternied peternied deleted the backport/backport-2375-to-2.x branch January 6, 2023 04:08
opensearch-trigger-bot bot added a commit that referenced this pull request Jan 6, 2023
peternied added a commit that referenced this pull request Jan 6, 2023
…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]>
opensearch-trigger-bot bot added a commit that referenced this pull request Jan 6, 2023
…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)
peternied added a commit to peternied/security that referenced this pull request Jan 10, 2023
…d` (opensearch-project#2378)

* [Backport 2.x] When excluding fields also exclude the term + `.keyword` (opensearch-project#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]>
wuychn pushed a commit to ochprince/security that referenced this pull request Mar 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 1.x backport to 1.x branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants