nasa / Common-Metadata-Repository

Apache License 2.0
337 stars 92 forks source link

CMR-9665: Add HTTP response event to access control app, indexer app, metadata-db app #2110

Closed zimzoom closed 4 months ago

zimzoom commented 4 months ago

Overview

What is the feature/fix?

Add remaining HTTP response event logs to access control app, indexer ap, and metadata db app. They were already existing in search and ingest and we originally had stated that after a couple months if we liked the logs they would be added to all.

What is the Solution?

Added necessary handlers

What areas of the application does this impact?

Logging

Checklist

codecov-commenter commented 4 months ago

Codecov Report

Attention: Patch coverage is 0% with 19 lines in your changes are missing coverage. Please review.

Project coverage is 57.81%. Comparing base (27b2aa8) to head (6d294f8). Report is 2 commits behind head on master.

Files Patch % Lines
...-app-lib/src/cmr/common_app/api/request_logger.clj 0.00% 9 Missing :warning:
...cess-control-app/src/cmr/access_control/routes.clj 0.00% 3 Missing :warning:
indexer-app/src/cmr/indexer/api/routes.clj 0.00% 3 Missing :warning:
metadata-db-app/src/cmr/metadata_db/api/routes.clj 0.00% 3 Missing :warning:
common-app-lib/src/cmr/common_app/config.clj 0.00% 0 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #2110 +/- ## =========================================== - Coverage 86.98% 57.81% -29.18% =========================================== Files 42 1041 +999 Lines 5884 70263 +64379 Branches 259 1976 +1717 =========================================== + Hits 5118 40620 +35502 - Misses 571 27779 +27208 - Partials 195 1864 +1669 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.