Skip to content

Conversation

@JeremyCloarec
Copy link
Contributor

@JeremyCloarec JeremyCloarec commented Oct 3, 2025

Proposed changes

  • add handling of too large elements in splitter: if lement has deps_number higher than configured max number, return it as too_large_element

opencti PR: OpenCTI-Platform/opencti#12632

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
  • I added/update the relevant documentation (either on github or on notion)
  • Where necessary I refactored code to improve the overall quality

Further comments

If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

@github-actions github-actions bot added the filigran team use to identify PR from the Filigran team label Oct 3, 2025
@JeremyCloarec JeremyCloarec added the multi-repository When a PR requires several repository changes label Oct 3, 2025
@xfournet xfournet added this to the Release 6.9.0 milestone Oct 6, 2025
@JeremyCloarec JeremyCloarec force-pushed the oob/inputSizeHardLimit branch from 1ede58f to 0450355 Compare October 8, 2025 09:27
@JeremyCloarec JeremyCloarec force-pushed the oob/inputSizeHardLimit branch from 0ea68b2 to ad7a8a8 Compare October 13, 2025 07:40
@xfournet xfournet self-requested a review October 14, 2025 21:39
@JeremyCloarec JeremyCloarec force-pushed the oob/inputSizeHardLimit branch from ad7a8a8 to e3f825d Compare October 15, 2025 09:57
@JeremyCloarec JeremyCloarec merged commit 36b07b3 into master Oct 16, 2025
9 checks passed
@JeremyCloarec JeremyCloarec deleted the oob/inputSizeHardLimit branch October 16, 2025 12:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

filigran team use to identify PR from the Filigran team multi-repository When a PR requires several repository changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants