Merge pull request #22 from christianlouis/copilot/check-black-formatting-errors

fix: apply black formatting to reports.py and dmarc_parser.py
This commit is contained in:
Christian Krakau-Louis
2026-03-29 13:06:14 +02:00
committed by GitHub
2 changed files with 4 additions and 12 deletions
+3 -9
View File
@@ -66,9 +66,7 @@ def _validate_upload_file(file: UploadFile, file_content: bytes) -> None:
"""
# Security: Validate filename is provided
if not file.filename:
raise HTTPException(
status_code=status.HTTP_400_BAD_REQUEST, detail="Filename is required"
)
raise HTTPException(status_code=status.HTTP_400_BAD_REQUEST, detail="Filename is required")
# Security: Validate file extension
file_ext = "." + file.filename.rsplit(".", 1)[-1].lower() if "." in file.filename else ""
@@ -97,13 +95,9 @@ def _handle_upload_value_error(filename: str, error_message: str) -> None:
status_code=status.HTTP_413_REQUEST_ENTITY_TOO_LARGE, detail="File too large"
)
elif "zip bomb" in error_message.lower():
raise HTTPException(
status_code=status.HTTP_400_BAD_REQUEST, detail="Invalid archive file"
)
raise HTTPException(status_code=status.HTTP_400_BAD_REQUEST, detail="Invalid archive file")
else:
raise HTTPException(
status_code=status.HTTP_400_BAD_REQUEST, detail="Invalid report format"
)
raise HTTPException(status_code=status.HTTP_400_BAD_REQUEST, detail="Invalid report format")
class UploadResponse(BaseModel):
+1 -3
View File
@@ -237,9 +237,7 @@ class DMARCParser:
# Log parse results for debugging
total_count = report["summary"]["total_count"]
logger.info(f"Parsed DMARC report for domain: {report.get('domain')}")
logger.info(
f"Found {len(records)} record entries with {total_count} total messages"
)
logger.info(f"Found {len(records)} record entries with {total_count} total messages")
logger.info(
f"Messages passed: {report['summary']['passed_count']}, "
f"failed: {report['summary']['failed_count']}"