Skip to content

update logging setup to respect json setting - #13216

Merged
SchrodingersGat merged 3 commits into
inventree:masterfrom
matmair:update-logging-setup
Nov 7, 2026
Merged

SchrodingersGat merged 3 commits into
inventree:masterfrom
matmair:update-logging-setup

Conversation

@matmair

@matmair matmair commented Nov 7, 2026

Copy link
Copy Markdown
Member

if JSON_LOG is true there is still a bunch of noise in logs which makes automated security and performance testing hard / noisy. This PR ensures more components are covered by the settings and (previously hardcoded) gunicorn logging can be switched off (we always recommend having a proxy in front. this info is often very useless.

@matmair matmair added this to the 1.8.0 milestone Nov 7, 2026
@matmair matmair self-assigned this Nov 7, 2026
@matmair matmair added CI CI / unit testing ecosystem setup Relates to the InvenTree setup / installation process labels Nov 7, 2026
@netlify

netlify Bot commented Nov 7, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for inventree-web-pui-preview canceled.

Name Link
🔨 Latest commit e8fa01d
🔍 Latest deploy log https://app.netlify.com/projects/inventree-web-pui-preview/deploys/6aee8760435994ba8a9af0a8

@codecov

codecov Bot commented Nov 7, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 57.14286% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 87.04%. Comparing base (0cb1852) to head (e8fa01d).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master   #13216   +/-   ##
=======================================
  Coverage   87.04%   87.04%           
=======================================
  Files        1568     1568           
  Lines      111588   111594    +6     
  Branches    12230    12230           
=======================================
+ Hits        97132    97138    +6     
  Misses      14390    14390           
  Partials       66       66           
Flag Coverage Δ
backend 91.37% <57.14%> (+0.02%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Components Coverage Δ
Backend Apps 92.89% <ø> (ø)
Backend General 94.96% <ø> (ø)
Frontend 79.69% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@SchrodingersGat
SchrodingersGat merged commit c54c69c into inventree:master Nov 7, 2026
51 checks passed
@SchrodingersGat
SchrodingersGat deleted the update-logging-setup branch November 7, 2026 05:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI CI / unit testing ecosystem setup Relates to the InvenTree setup / installation process

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants