Skip to content

[SEKOIA.IO] update cache system in Sekoia connector#5367

Merged
jabesq merged 2 commits intoOpenCTI-Platform:masterfrom
SEKOIA-IO:feat/optimize-sekoia-connector
Feb 9, 2026
Merged

[SEKOIA.IO] update cache system in Sekoia connector#5367
jabesq merged 2 commits intoOpenCTI-Platform:masterfrom
SEKOIA-IO:feat/optimize-sekoia-connector

Conversation

@mchupeau-sk
Copy link
Copy Markdown
Contributor

@mchupeau-sk mchupeau-sk commented Dec 8, 2025

Proposed changes

  • Optimize the number of request to Sekoia.io API using cache system

Related issues

Checklist

  • I consider the submitted work as finished
  • I have signed my commits using GPG key.
  • I tested the code for its functionality using different use cases
  • I added/update the relevant documentation (either on github or on notion)
  • Where necessary I refactored code to improve the overall quality

Further comments

Copy link
Copy Markdown

@squioc squioc left a comment

Choose a reason for hiding this comment

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

LGTM

@romain-filigran romain-filigran added the partner used to identify PR from patner label Dec 9, 2025
@mchupeau-sk mchupeau-sk marked this pull request as ready for review December 9, 2025 12:00
@romain-filigran romain-filigran added the filigran verify Use to identify PR of connector Verified label Dec 12, 2025
@SamuelHassine SamuelHassine force-pushed the master branch 2 times, most recently from a35f198 to 34b8e57 Compare January 9, 2026 14:33
@helene-nguyen helene-nguyen linked an issue Jan 29, 2026 that may be closed by this pull request
@helene-nguyen
Copy link
Copy Markdown
Member

@mchupeau-sk
To complete our review, we need our Intelligence subscription to be re-activated :)

@mchupeau-sk
Copy link
Copy Markdown
Contributor Author

Hello @helene-nguyen,

Normally it's good from our side :)

Copy link
Copy Markdown
Member

@jabesq jabesq left a comment

Choose a reason for hiding this comment

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

LGTM 👍
Tested with a normal cache size and a very small one to validate the mechanism. It works as expected.

@jabesq jabesq merged commit b81cadc into OpenCTI-Platform:master Feb 9, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

filigran verify Use to identify PR of connector Verified partner used to identify PR from patner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[sekoia.io] Duplicate calls to identities

7 participants