Skip to content
This repository has been archived by the owner on Dec 6, 2024. It is now read-only.

fix: add attributes for better logging #897

Merged
merged 3 commits into from
Feb 2, 2022
Merged

fix: add attributes for better logging #897

merged 3 commits into from
Feb 2, 2022

Conversation

SanketD92
Copy link
Contributor

Issue #, if available:
V377389478

Description of changes:
The log entries now include the time, remote client IP address, user role, and any associated usernames.

Checklist:

  • Have you successfully deployed to an AWS account with your changes?
  • Have you successfully tested with your changes locally?

AS review ticket id:

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@SanketD92 SanketD92 requested a review from a team as a code owner February 2, 2022 17:13
@github-actions github-actions bot added the size/s label Feb 2, 2022
@SanketD92 SanketD92 changed the base branch from mainline to develop February 2, 2022 17:14
@codecov
Copy link

codecov bot commented Feb 2, 2022

Codecov Report

Merging #897 (75c7d05) into develop (4c652a2) will decrease coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #897      +/-   ##
===========================================
- Coverage    51.14%   51.11%   -0.03%     
===========================================
  Files          292      292              
  Lines        16427    16435       +8     
  Branches      2547     2551       +4     
===========================================
  Hits          8401     8401              
- Misses        7054     7058       +4     
- Partials       972      976       +4     
Impacted Files Coverage Δ
lib/audit/audit-writer-service.js 8.82% <0.00%> (-2.72%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4c652a2...75c7d05. Read the comment docs.

@SanketD92 SanketD92 merged commit 0a3ea5c into develop Feb 2, 2022
@SanketD92 SanketD92 deleted the fix-audit-logs branch February 2, 2022 19:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants