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
self.opencti.work.report_expectation(
work_id,
{
"error": "Incompatible element in bundle",
Copy link
Member

Choose a reason for hiding this comment

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

Too large element in bundle ?

"error": "Incompatible element in bundle",
"source": "Element "
+ too_large_elements_bundle["id"]
+ " is incompatible and couldn't be processed",
Copy link
Member

Choose a reason for hiding this comment

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

is too large and couldn't be processed

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.

3 participants