Skip to content

Add javadoc about ThreadContextPermission for stashWithOrigin and stashAndMergeHeaders#15051

Merged
reta merged 1 commit intoopensearch-project:mainfrom
cwperks:stash-javadoc
Aug 1, 2024
Merged

Add javadoc about ThreadContextPermission for stashWithOrigin and stashAndMergeHeaders#15051
reta merged 1 commit intoopensearch-project:mainfrom
cwperks:stash-javadoc

Conversation

@cwperks
Copy link
Copy Markdown
Member

@cwperks cwperks commented Jul 31, 2024

Description

Follow-up to #15039 to add javadoc on the permissions added to ThreadContext.stashWithOrigin and ThreadContext.stashAndMergeHeaders

Check List

  • Functionality includes testing.
  • API changes companion pull request created, if applicable.
  • Public documentation issue/PR created, if applicable.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

…shAndMergeHeaders

Signed-off-by: Craig Perkins <cwperx@amazon.com>
@cwperks
Copy link
Copy Markdown
Member Author

cwperks commented Jul 31, 2024

@reta Can we add skip-changelog to this PR? I don't think it needs a CHANGELOG entry.

@reta reta added skip-changelog backport 2.x Backport to 2.x branch labels Jul 31, 2024
@github-actions
Copy link
Copy Markdown
Contributor

✅ Gradle check result for c13441c: SUCCESS

@codecov
Copy link
Copy Markdown

codecov Bot commented Jul 31, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 71.88%. Comparing base (0324edd) to head (c13441c).

Additional details and impacted files
@@             Coverage Diff              @@
##               main   #15051      +/-   ##
============================================
- Coverage     71.90%   71.88%   -0.03%     
+ Complexity    62828    62820       -8     
============================================
  Files          5169     5169              
  Lines        294645   294645              
  Branches      42610    42610              
============================================
- Hits         211869   211809      -60     
- Misses        65343    65459     +116     
+ Partials      17433    17377      -56     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@reta reta merged commit 67a2e4c into opensearch-project:main Aug 1, 2024
opensearch-trigger-bot Bot pushed a commit that referenced this pull request Aug 1, 2024
…shAndMergeHeaders (#15051)

Signed-off-by: Craig Perkins <cwperx@amazon.com>
(cherry picked from commit 67a2e4c)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
reta pushed a commit that referenced this pull request Aug 1, 2024
…shAndMergeHeaders (#15051) (#15055)

(cherry picked from commit 67a2e4c)

Signed-off-by: Craig Perkins <cwperx@amazon.com>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
harshavamsi pushed a commit to harshavamsi/OpenSearch that referenced this pull request Aug 20, 2024
…shAndMergeHeaders (opensearch-project#15051)

Signed-off-by: Craig Perkins <cwperx@amazon.com>
wdongyu pushed a commit to wdongyu/OpenSearch that referenced this pull request Aug 22, 2024
…shAndMergeHeaders (opensearch-project#15051)

Signed-off-by: Craig Perkins <cwperx@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport 2.x Backport to 2.x branch skip-changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants