Skip to content

IEBH-458: Add audit logs#19

Merged
vmoshynskyi merged 1 commit intomainfrom
IEBH-458
Nov 14, 2025
Merged

IEBH-458: Add audit logs#19
vmoshynskyi merged 1 commit intomainfrom
IEBH-458

Conversation

@vmoshynskyi
Copy link
Member

  • Add audit logs.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds comprehensive audit logging to the upload service to track file upload operations and improves dependency management. The audit logs capture key events throughout the upload lifecycle including job preparation, chunk processing, and both successful completions and unexpected errors.

  • Updated pilot-platform-common dependency from version 0.3.0 to 0.8.2 to support the new audit logging functionality
  • Added audit logs at critical points in the upload workflow to track user actions
  • Restricted Python version to 3.10.x to match dependency requirements

Reviewed Changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
pyproject.toml Bumped service version to 2.2.16, updated Python constraint to 3.10.x, and upgraded pilot-platform-common to 0.8.2
poetry.lock Updated dependency lock file reflecting pilot-platform-common upgrade and associated dependency changes (redis replaces aioredis, cryptography removed)
app/logger.py Added Logger type import and annotation for enhanced audit logging support
app/routers/v1/api_data_upload.py Added audit log calls at key points: job preparation start/success/error, chunk finalization submission, chunk preparation start/success/error
tests/conftest.py Added app fixture parameter to mock_boto3 to ensure proper initialization order

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions
Copy link

Coverage

Coverage Report
FileStmtsMissCoverMissing
__init__.py00100% 
__main__.py050%7, 9, 11–13
api_registry.py00100% 
config.py00100% 
logger.py00100% 
main.py0783%80–81, 83–85, 87, 91
commons
   __init__.py00100% 
   kafka_producer.py02744%32–39, 41, 48–50, 62–66, 82–84, 86, 88, 108, 110, 126–127, 129
commons/data_providers
   __init__.py00100% 
   redis.py01660%28–29, 37, 40, 43, 46, 49–51, 54, 57, 60–64
   redis_project_session_job.py02470%66, 71, 73, 75, 88, 91–96, 203–204, 210–215, 217–220, 222
components
   __init__.py00100% 
components/request
   __init__.py00100% 
   network.py00100% 
models
   __init__.py00100% 
   base_models.py00100% 
   folder.py0890%101, 117, 126–131
   models_item.py0185%23
   models_resumable_upload.py00100% 
   models_upload.py00100% 
resources
   __init__.py00100% 
   archive_file_type_mapping.py00100% 
   decorator.py00100% 
   error_handler.py01669%34–40, 50–57, 82
   health_check.py02921%25–29, 31–35, 49–50, 52–54, 56–58, 60–63, 65, 78–81, 83–84
   helpers.py01887%26, 31, 36, 138–140, 148–150, 158–160, 168–170, 197–199
   lock.py0250%7, 9, 12–13, 16–22, 24, 27–28, 31–32, 35–36, 38–43, 45
routers
   __init__.py00100% 
   api_root.py0673%40–41, 43, 52–54
   exceptions.py0983%24, 31, 38, 54, 58, 62, 92, 96, 100
routers/v1
   __init__.py00100% 
   api_data_upload.py07376%120–122, 261–262, 267–268, 325–327, 329, 335–336, 338–339, 362, 364–365, 367–373, 375, 521–523, 604, 662–666, 668–669, 675–678, 698, 700–702, 711–714, 716–717, 719, 737–740, 750–753, 755–756, 764, 769–778
   exceptions.py00100% 
routers/v1/api_resumable_upload
   __init__.py00100% 
   api_resumable_upload.py0392%50–52
   utils.py0625%11–14, 16, 24
TOTAL118727377% 

@vmoshynskyi vmoshynskyi merged commit 3e7d3c5 into main Nov 14, 2025
12 checks passed
@vmoshynskyi vmoshynskyi deleted the IEBH-458 branch November 14, 2025 14:34
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.

1 participant