Supporting more use cases for the log_context decorator

[galvana]

Closes LJ-488

Description Of Changes

Fixing bug where positional arguments were not being captured correctly by the @log_context decorator.

Code Changes

• Fixed positional arguments use case and added more test coverage

Steps to Confirm

1. Submit a privacy request, you should see the privacy_request_id as extra context for the following log

fides  | 2025-03-11 01:03:19.697 | INFO     | fides.service.privacy_request.privacy_request_service:queue_privacy_request:512 - Queueing privacy request from step None | {'privacy_request_id': 'pri_003425fe-a37e-4560-967e-58b3c0182c84'}

Pre-Merge Checklist

• Issue requirements met
• All CI pipelines succeeded
• CHANGELOG.md updated
  • Add a db-migration This indicates that a change includes a database migration label to the entry if your change includes a DB migration
  • Add a high-risk This issue suggests changes that have a high-probability of breaking existing code label to the entry if your change includes a high-risk change (i.e. potential for performance impact or unexpected regression) that should be flagged
• Followup issues:
  • Followup issues created (include link)
  • No followup issues
• Database migrations:
  • Ensure that your downrev is up to date with the latest revision on main
  • Ensure that your downgrade() migration is correct and works
    • If a downgrade migration is not possible for this change, please call this out in the PR description!
  • No migrations
• Documentation:
  • Documentation complete, PR opened in fidesdocs
  • Documentation issue created in fidesdocs
  • If there are any new client scopes created as part of the pull request, remember to update public-facing documentation that references our scope registry
  • No documentation updates required

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
fides-plus-nightly	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Mar 11, 2025 7:37pm
fides-privacy-center	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Mar 11, 2025 7:37pm

[galvana]

This typo is why I missed it the first time around 😵‍💫

[codecov]

Codecov Report

Attention: Patch coverage is 82.35294% with 3 lines in your changes missing coverage. Please review.

Project coverage is 87.01%. Comparing base (728a36a) to head (34cea85).

Files with missing lines	Patch %	Lines
src/fides/api/util/logger_context_utils.py	82.35%	2 Missing and 1 partial ⚠️

❌ Your patch check has failed because the patch coverage (82.35%) is below the target coverage (100.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #5866      +/-   ##
==========================================
- Coverage   87.02%   87.01%   -0.01%     
==========================================
  Files         407      407              
  Lines       25125    25142      +17     
  Branches     2706     2712       +6     
==========================================
+ Hits        21864    21878      +14     
- Misses       2678     2680       +2     
- Partials      583      584       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[erosselli]

🤩 Very cool

[cypress]

fides    Run #12658

Run Properties:   Passed #12658  •  660bd4ddab: Supporting more use cases for the log_context decorator (#5866)

Project	fides
Branch Review	main
Run status	Passed #12658
Run duration	00m 51s
Commit	660bd4ddab: Supporting more use cases for the log_context decorator (#5866)
Committer	Adrian Galvan
View all properties for this run ↗︎

---

Test results
Failures	0
Flaky	0
Pending	0
Skipped	0
Passing	5
View all changes introduced in this branch ↗︎