Skip to content

[group-ib] Fix arbitrary user id same as #2507#6260

Open
leitosama wants to merge 1 commit into
OpenCTI-Platform:masterfrom
leitosama:arbitrary_uid_group-ib-upstream
Open

[group-ib] Fix arbitrary user id same as #2507#6260
leitosama wants to merge 1 commit into
OpenCTI-Platform:masterfrom
leitosama:arbitrary_uid_group-ib-upstream

Conversation

@leitosama
Copy link
Copy Markdown
Contributor

@leitosama leitosama commented Apr 18, 2026

This PR fixes errors cause by OCP running container using an arbitrarily assigned user ID (I fixed same errors in other connectors in #2507).

group-ib uses file as cache storage (

@cache_data(
cache_dir=ConfigConnector.MITRE_CACHE_FOLDER,
cache_file=ConfigConnector.MITRE_CACHE_FILENAME,
ttl=1,
).

Proposed changes

  • Change group for /opt/connector directory to root
  • Modify group permissions for /opt/connector directory

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

@filigran-cla-bot filigran-cla-bot Bot added the cla:pending CLA signature required label Apr 18, 2026
@filigran-cla-bot
Copy link
Copy Markdown

filigran-cla-bot Bot commented Apr 18, 2026

Contributor License Agreement

CLA signed 💚

Thank you @leitosama for signing the Contributor License Agreement! Your pull request can now be reviewed and merged.

We appreciate your contribution to Filigran's open source projects! ❤️

This is an automated message from the Filigran CLA Bot.

@filigran-cla-bot filigran-cla-bot Bot removed the cla:pending CLA signature required label Apr 18, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 18, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 0.09%. Comparing base (7292475) to head (4363021).

❗ There is a different number of reports uploaded between BASE (7292475) and HEAD (4363021). Click for more details.

HEAD has 91 uploads less than BASE
Flag BASE (7292475) HEAD (4363021)
connectors 92 1
Additional details and impacted files
@@            Coverage Diff             @@
##           master   #6260       +/-   ##
==========================================
- Coverage   40.08%   0.09%   -39.99%     
==========================================
  Files        2129    1658      -471     
  Lines      127662   99849    -27813     
==========================================
- Hits        51169      94    -51075     
- Misses      76493   99755    +23262     
Flag Coverage Δ
baseline 0.00% <ø> (ø)
connectors 96.90% <ø> (+21.83%) ⬆️

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:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@romain-filigran romain-filigran added the community use to identify PR from community label Apr 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community use to identify PR from community

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[import-document,import-file-stix] Support running as an arbitrary user (OpenShift Container Platform)

3 participants