Skip to content

[client] move pythonjsonlogger.jsonlogger to pythonjsonlogger.json (#13114)#13113

Merged
xfournet merged 1 commit into
masterfrom
feature/1027-move-pythonjsonlogger.jsonlogger-to-pythonjsonlogger.json
Nov 13, 2025
Merged

[client] move pythonjsonlogger.jsonlogger to pythonjsonlogger.json (#13114)#13113
xfournet merged 1 commit into
masterfrom
feature/1027-move-pythonjsonlogger.jsonlogger-to-pythonjsonlogger.json

Conversation

@mariot

@mariot mariot commented Nov 13, 2025

Copy link
Copy Markdown
Member

Proposed changes

  • move pythonjsonlogger.jsonlogger to pythonjsonlogger.json per the docs

Related issues

Checklist

  • I consider the submitted work as finished
  • I tested the code for its functionality
  • I wrote test cases for the relevant uses case (coverage and e2e)
  • I added/update the relevant documentation (either on github or on notion)
  • Where necessary I refactored code to improve the overall quality

Further comments

@mariot mariot added feature Type: new feature or capability (feat:). filigran team Item from the Filigran team. tiny Tiny change (less than a week to solve). labels Nov 13, 2025
@codecov

codecov Bot commented Nov 13, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 16.20%. Comparing base (260712f) to head (4ee42c6).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master   #13113   +/-   ##
=======================================
  Coverage   16.20%   16.20%           
=======================================
  Files        2842     2842           
  Lines      411830   411830           
  Branches    11400    11401    +1     
=======================================
+ Hits        66719    66721    +2     
+ Misses     345111   345109    -2     
Flag Coverage Δ
opencti 16.20% <ø> (+<0.01%) ⬆️
opencti-front 3.89% <ø> (+<0.01%) ⬆️
opencti-graphql 67.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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@aHenryJard aHenryJard left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please rebase, the client python CI was not ready when you did this PR.

@aHenryJard aHenryJard changed the title [deprecation] move pythonjsonlogger.jsonlogger to pythonjsonlogger.json [client] move pythonjsonlogger.jsonlogger to pythonjsonlogger.json (#13114) Nov 13, 2025
@mariot mariot force-pushed the feature/1027-move-pythonjsonlogger.jsonlogger-to-pythonjsonlogger.json branch from e1806e4 to abb251a Compare November 13, 2025 13:43
@github-actions

Copy link
Copy Markdown

Thank you for your contribution, but we need you to sign your commits. Please see https://docs.github.com/en/authentication/managing-commit-signature-verification/signing-commits

1 similar comment
@github-actions

Copy link
Copy Markdown

Thank you for your contribution, but we need you to sign your commits. Please see https://docs.github.com/en/authentication/managing-commit-signature-verification/signing-commits

@mariot mariot force-pushed the feature/1027-move-pythonjsonlogger.jsonlogger-to-pythonjsonlogger.json branch from abb251a to 4ee42c6 Compare November 13, 2025 13:47
@xfournet xfournet merged commit e2ed23e into master Nov 13, 2025
13 checks passed
@xfournet xfournet deleted the feature/1027-move-pythonjsonlogger.jsonlogger-to-pythonjsonlogger.json branch November 13, 2025 14:31
@SamuelHassine SamuelHassine removed the feature Type: new feature or capability (feat:). label Jun 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

filigran team Item from the Filigran team. tiny Tiny change (less than a week to solve).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[deprecation] pythonjsonlogger.jsonlogger has been moved to pythonjsonlogger.json

4 participants