Skip to content

[Backport 2.x] deserializeSafeFromHeader uses context.getHeader(headerName) instead of context.getHeaders()#2771

Merged
peternied merged 1 commit into2.xfrom
backport/backport-2768-to-2.x
May 15, 2023
Merged

[Backport 2.x] deserializeSafeFromHeader uses context.getHeader(headerName) instead of context.getHeaders()#2771
peternied merged 1 commit into2.xfrom
backport/backport-2768-to-2.x

Conversation

@opensearch-trigger-bot
Copy link
Contributor

Backport 7c4e06d from #2768

…ead of `context.getHeaders()` (#2768)

(cherry picked from commit 7c4e06d)
@codecov
Copy link

codecov bot commented May 13, 2023

Codecov Report

Merging #2771 (a58ad88) into 2.x (cbb0725) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##                2.x    #2771   +/-   ##
=========================================
  Coverage     61.32%   61.32%           
  Complexity     3312     3312           
=========================================
  Files           268      268           
  Lines         18346    18342    -4     
  Branches       3246     3245    -1     
=========================================
- Hits          11250    11249    -1     
+ Misses         5523     5521    -2     
+ Partials       1573     1572    -1     
Impacted Files Coverage Δ
.../org/opensearch/security/support/HeaderHelper.java 60.00% <100.00%> (-3.16%) ⬇️

... and 1 file with indirect coverage changes

@willyborankin
Copy link
Collaborator

Tests failures related to 10 seconds which we fixed in #2758

@peternied peternied merged commit dae8f76 into 2.x May 15, 2023
@peternied peternied deleted the backport/backport-2768-to-2.x branch May 15, 2023 21:20
stephen-crawford pushed a commit to stephen-crawford/security that referenced this pull request May 16, 2023
…ead of `context.getHeaders()` (opensearch-project#2768) (opensearch-project#2771)

(cherry picked from commit 7c4e06d)

Co-authored-by: Paras Jain <parasjain.jain@outlook.com>
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.

5 participants