Repository navigation
fix(apollo-vertex): keep the user token out of solution-test trigger bodies - #1434
Conversation
…bodies The trigger body becomes the job's input arguments, which anyone with Jobs.View in the folder can read. Send the bearer token in the Authorization header only; the backend reads the caller from the job's creator (UiPath/fins-vertical-solution#1502). Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01VQA1xEsDwn4t2UBBRpTroi
There was a problem hiding this comment.
🔵 Needs a closer look
Compatibility and authorization depend on the linked FINS backend change, which requires human verification.
No open issues 🎉
What changed in this PR
Prevents solution-test actions from exposing bearer tokens through Orchestrator job input arguments.
Changes:
- Keeps bearer tokens exclusively in the
Authorizationheader. - Removes body authentication fields, relying on the coordinated FINS backend change to identify callers.
| File | Description |
|---|---|
| packages/apollo-vertex/registry/solution-tests/create-actions.ts | Removes token-bearing request fields and documents header-only authentication. |
Review effort: Balanced
Dependency License Review
License distribution
Excluded packages
|
📊 Coverage + size by packagePer-package coverage and bundle size on this PR. New-line coverage = of the source lines this PR adds or changes, the % hit by tests.
"Coverage" is each package's own |
Storybook visual diff✅ No stories are affected by this PR's changes; nothing to compare. Logs Updated (PT): Oct 21, 2026, 07:45:25 AM |
The solution-tests write actions copied the user's bearer token into the HTTP trigger's request body. Orchestrator stores that body as the job's input arguments, which anyone with Jobs.View in the folder can read.
create-actions.tsnow sends the token in theAuthorizationheader only.The FINS backend gets the caller from the job's Orchestrator creator instead: UiPath/fins-vertical-solution#1502. FINS is the only consumer of solution-tests, so the old body fields go without a compatibility path. The file matches the FINS webapps' copies byte for byte.
Checked with
tsc --noEmit,oxlint --type-aware registry/solution-testsandbiome formatinpackages/apollo-vertex.🤖 Generated with Claude Code
https://claude.ai/code/session_01VQA1xEsDwn4t2UBBRpTroi