feat(oracle-epm): add guarded foundation - #7469
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
️✅ There are no secrets present in this pull request anymore.If these secrets were true positive and are still valid, we highly recommend you to revoke them. 🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request. |
Greptile SummaryAdds a guarded Oracle EPM foundation without introducing a product-facing integration.
Confidence Score: 5/5The PR appears safe to merge. No blocking failure remains.
|
| Filename | Overview |
|---|---|
| apps/sim/lib/internal/oracle-epm/files.server.ts | Implements bounded file streaming and serialized cancellation cleanup; the previously reported completion and presigning races are addressed. |
| apps/sim/lib/internal/oracle-epm/jobs.ts | Implements deadline-aware polling and races non-cooperative reads against the combined abort signal, addressing the previous thread. |
| apps/sim/lib/internal/oracle-epm/client.server.ts | Adds a fixed-origin, SSRF-guarded client with bounded requests, responses, retries, and opaque returned-link capabilities. |
| apps/sim/lib/internal/oracle-epm/endpoint.ts | Adds validated and frozen endpoint declarations for child-owned Oracle EPM operations. |
| apps/sim/lib/credentials/client-credential-accounts/minters/oracle-epm.ts | Adds local Basic-auth minting with normalized environment binding and credential-text validation. |
Flowchart
%%{init: {'theme': 'neutral'}}%%
flowchart LR
C[Oracle EPM credential] --> M[Local Basic-auth minter]
M --> G[Fixed-destination guarded client]
G --> E[Branded endpoint declarations]
G --> L[Validated returned-link capabilities]
G --> J[Bounded job polling]
G --> F[Bounded file streaming and storage]
Reviews (9): Last reviewed commit: "refactor(oracle-epm): keep API contract ..." | Re-trigger Greptile
|
@cubic-dev-ai review this PR |
@BillLeoutsakosvl346 I have started the AI code review. It will take a few minutes to complete. |
ff77e58 to
7ba3213
Compare
|
@cubic-dev-ai review this PR |
@BillLeoutsakosvl346 I have started the AI code review. It will take a few minutes to complete. |
|
@cubic-dev-ai review this PR |
@BillLeoutsakosvl346 I have started the AI code review. It will take a few minutes to complete. |
|
@cubic-dev-ai review this PR |
@BillLeoutsakosvl346 I have started the AI code review. It will take a few minutes to complete. |
|
@cubic-dev-ai review this PR |
@BillLeoutsakosvl346 I have started the AI code review. It will take a few minutes to complete. |
There was a problem hiding this comment.
No issues found across 34 files
Confidence score: 5/5
- Automated review surfaced no issues in the provided summaries.
- No files require special attention.
You've manually re-run cubic several times on this PR. Each manual re-review checks the full PR again and counts toward your usage quota. To preserve your usage limits, we recommend letting cubic automatically review new commits.
Re-trigger cubic
|
@cubic-dev-ai review this PR |
@BillLeoutsakosvl346 I have started the AI code review. It will take a few minutes to complete. |
There was a problem hiding this comment.
No issues found across 30 files
Confidence score: 5/5
- Automated review surfaced no issues in the provided summaries.
- No files require special attention.
You've manually re-run cubic several times on this PR. Each manual re-review checks the full PR again and counts toward your usage quota. To preserve your usage limits, we recommend letting cubic automatically review new commits.
Re-trigger cubic
|
@cubic-dev-ai review this PR |
@BillLeoutsakosvl346 I have started the AI code review. It will take a few minutes to complete. |
There was a problem hiding this comment.
No issues found across 26 files
Confidence score: 5/5
- Automated review surfaced no issues in the provided summaries.
- No files require special attention.
You've manually re-run cubic several times on this PR. Each manual re-review checks the full PR again and counts toward your usage quota. To preserve your usage limits, we recommend letting cubic automatically review new commits.
Re-trigger cubic
Summary
Type of Change
Testing
Deferred Platform Work
Checklist