Skip to content

feat: add client inventory logging for compliance#45

Merged
flacatus merged 1 commit into
konflux-ci:mainfrom
msu8:mcp_client_inventory_clean
Mar 3, 2026
Merged

feat: add client inventory logging for compliance#45
flacatus merged 1 commit into
konflux-ci:mainfrom
msu8:mcp_client_inventory_clean

Conversation

@msu8

@msu8 msu8 commented Feb 10, 2026

Copy link
Copy Markdown
Contributor
  • Log client_id, hashed_ip, user_agent on /mcp requests
  • Link client_id to tool call logs for traceability, as compliance requires "identify the hosts for all actions"
  • Add PVC for persistent log storage of size 5Gi - might be an overkill.

- Log client_id, hashed_ip, user_agent on /mcp requests
- Link client_id to tool call logs for traceability
- Add PVC for persistent log storage
@snyk-io

snyk-io Bot commented Feb 10, 2026

Copy link
Copy Markdown
Contributor

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 29.41176% with 24 lines in your changes missing coverage. Please review.
✅ Project coverage is 51.63%. Comparing base (bef5536) to head (2ea1ee0).
⚠️ Report is 46 commits behind head on main.

Files with missing lines Patch % Lines
server/transport/http_transport.py 4.54% 21 Missing ⚠️
utils/logger.py 75.00% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #45      +/-   ##
==========================================
+ Coverage   47.54%   51.63%   +4.09%     
==========================================
  Files          27       38      +11     
  Lines        1424     2868    +1444     
  Branches      178      387     +209     
==========================================
+ Hits          677     1481     +804     
- Misses        724     1326     +602     
- Partials       23       61      +38     
Flag Coverage Δ
unit-tests 51.63% <29.41%> (+4.09%) ⬆️

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

☔ 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.

@flacatus

Copy link
Copy Markdown
Member

/ok-to-test

@flacatus flacatus merged commit 19f69c8 into konflux-ci:main Mar 3, 2026
9 checks passed
msu8 added a commit to msu8/infra-common-deployments that referenced this pull request Mar 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants