Skip to content

[Backport 2.x] Honor log_request_body setting in compliance audit log #4918

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

Merged
merged 4 commits into from
Nov 20, 2024

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 5698a10 from #4832.

Signed-off-by: Craig Perkins <[email protected]>
(cherry picked from commit 5698a10)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
RyanL1997
RyanL1997 previously approved these changes Nov 19, 2024
Signed-off-by: Craig Perkins <[email protected]>
@cwperks cwperks dismissed stale reviews from DarshitChanpura and RyanL1997 via 06f8dd6 November 19, 2024 20:22
Signed-off-by: Craig Perkins <[email protected]>
Copy link

codecov bot commented Nov 19, 2024

Codecov Report

Attention: Patch coverage is 53.33333% with 7 lines in your changes missing coverage. Please review.

Project coverage is 63.89%. Comparing base (70febbd) to head (3451cec).
Report is 1 commits behind head on 2.x.

Files with missing lines Patch % Lines
...earch/security/auditlog/impl/AbstractAuditLog.java 53.33% 3 Missing and 4 partials ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##              2.x    #4918      +/-   ##
==========================================
+ Coverage   63.87%   63.89%   +0.02%     
==========================================
  Files         330      330              
  Lines       23142    23144       +2     
  Branches     3750     3751       +1     
==========================================
+ Hits        14783    14789       +6     
- Misses       6523     6524       +1     
+ Partials     1836     1831       -5     
Files with missing lines Coverage Δ
...earch/security/auditlog/impl/AbstractAuditLog.java 76.73% <53.33%> (+0.10%) ⬆️

... and 5 files with indirect coverage changes

---- 🚨 Try these New Features:

@DarshitChanpura DarshitChanpura merged commit a1ff971 into 2.x Nov 20, 2024
89 of 91 checks passed
@DarshitChanpura DarshitChanpura deleted the backport/backport-4832-to-2.x branch November 20, 2024 01:20
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.

3 participants