Skip to content

fix(compose): give the XTM One worker its public URL and accept the platforms' internal audience (#47) - #48

Merged
Samuel Hassine (SamuelHassine) merged 2 commits into
mainfrom
fix/xtm-one-internal-urls-47
Sep 29, 2026
Merged

Samuel Hassine (SamuelHassine) merged 2 commits into
mainfrom
fix/xtm-one-internal-urls-47

Conversation

@SamuelHassine

Copy link
Copy Markdown
Member

Proposed changes

Related issues

How to test this PR

  1. docker compose -f docker-compose.yml --env-file .env.sample config -q (passes).
  2. With the product fixes released: docker compose up -d, then an Ask Ariane question on OpenCTI data and on OpenAEV data, and an XTM One assignment that calls OpenCTI (worker path).

Checklist

  • The PR title follows the Conventional Commits convention type(scope?): description (#issue)
  • I signed my commits
  • This PR is linked to an issue
  • I consider the submitted work as finished

…latforms' internal audience (#47)

The worker signed the tokens of background work with XTM One's built-in default issuer, which OpenCTI and OpenAEV never trust. XTM One now also accepts http://xtm-one:4000 as the audience of their tokens, for platform versions that address them to the internal URL. The README explains the public and internal URLs.
Copilot AI balanced review requested due to automatic review settings September 29, 2026 13:40
@SamuelHassine Samuel Hassine (SamuelHassine) added filigran team vibe-coded PR: AI-assisted change — the author reviews it before requesting others' review. labels Sep 29, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

The README links issue reports instead of the required product-fix PRs, making compatible versions unclear.

Review effort: Balanced
Findings: 1 Low severity

Open (1)
What changed in this PR

Updates Compose authentication URLs so XTM One integrations can trust signed cross-platform requests.

Changes:

  • Configures the worker with XTM One’s public URL.
  • Accepts the platforms’ internal XTM One audience.
  • Documents public versus internal URLs.
File Description
README.md Documents URL identities and compatibility requirements.
docker-compose.yml Adds worker URLs and accepted token audience.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread README.md Outdated

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

The changes satisfy the linked issue and match the corresponding product-stack fixes.

Review effort: Balanced
Findings: None

Resolved since last review (1)

@SamuelHassine
Samuel Hassine (SamuelHassine) merged commit 76657cc into main Sep 29, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

filigran team vibe-coded PR: AI-assisted change — the author reviews it before requesting others' review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(compose): give the XTM One worker its public URL and accept the platforms' internal audience

2 participants