search

stellar / django-polaris

An extendable Django app for building modular Stellar services
https://django-polaris.readthedocs.io
Apache License 2.0
95 stars 70 forks source link

update logging format #610

Closed JakeUrban closed 2 years ago

JakeUrban commented 2 years ago

Updates the log message format to include the logger's name (which by convention is equal to the __name__ python variable) and the line number of the log statement. This also removes the unnecessary subclass of LogAdapter.

An unrelated change updates the polaris/settings.py file's MIDDLEWARE list to match the reference server's list.

codecov-commenter commented 2 years ago

Codecov Report

Merging #610 (2ff6c77) into master (5a207b9) will increase coverage by 0.20%. The diff coverage is 100.00%.

:exclamation: Current head 2ff6c77 differs from pull request most recent head 1f5aabc. Consider uploading reports for the commit 1f5aabc to get more accurate results

@@            Coverage Diff             @@
##           master     #610      +/-   ##
==========================================
+ Coverage   83.82%   84.03%   +0.20%     
==========================================
  Files          52       52              
  Lines        4382     4377       -5     
==========================================
+ Hits         3673     3678       +5     
+ Misses        709      699      -10
Impacted Files Coverage Δ
polaris/polaris/utils.py 87.00% <100.00%> (-0.26%) :arrow_down:
polaris/polaris/middleware.py 90.90% <0.00%> (+90.90%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 5a207b9...1f5aabc. Read the comment docs.