Correlation ID - Part 1

[aqeelat]

feat: Added support for storing correlation ids passed to a custom header

[codecov]

Codecov Report

Merging #481 (5238a70) into master (a733cd0) will increase coverage by 0.41%.
The diff coverage is 98.50%.

@@            Coverage Diff             @@
##           master     #481      +/-   ##
==========================================
+ Coverage   92.83%   93.24%   +0.41%     
==========================================
  Files          27       29       +2     
  Lines         795      859      +64     
==========================================
+ Hits          738      801      +63     
- Misses         57       58       +1     

Impacted Files	Coverage Δ
auditlog/filters.py	91.66% <90.90%> (-0.65%)	⬇️
auditlog/admin.py	100.00% <100.00%> (ø)
auditlog/cid.py	100.00% <100.00%> (ø)
auditlog/conf.py	100.00% <100.00%> (ø)
auditlog/middleware.py	100.00% <100.00%> (ø)
auditlog/migrations/0014_logentry_cid.py	100.00% <100.00%> (ø)
auditlog/mixins.py	89.28% <100.00%> (+1.40%)	⬆️
auditlog/models.py	87.96% <100.00%> (+0.25%)	⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[hramezani]

@aqeelat Thanks for this. this is a good starting point 👍

Please remember to rebase the patch and re-run the makemigrations command (because I have merged #478)

I will take a look later this week

[aqeelat]

@hramezani updated.

[hramezani]

Thanks @aqeelat for the update.
I left some comments.

Please add documentation for the new settings and also add tests for checking the saved cid in the model.

[aqeelat]

@hramezani done

[hramezani]

Thanks @aqeelat for updating the patch.
I've left some comments. Please address them.

Please also add test to proved saved cid value in model

[aqeelat]

Thanks @aqeelat for updating the patch.

I've left some comments. Please address them.

Please also add test to proved saved cid value in model

I added the test already. It should be in the same middleware test. I compared the value in the history with the expected value.

[hramezani]

I think everything is done. you just need to update the comment in cid.py. Then we are good to go

[aqeelat]

@hramezani all is done now. LMK if you need anything else.