Repository navigation
fix(credentials): Splunk password sign-in, Snowflake key pair in the CLI, Apollo OAuth people search, and one short SailPoint throttle wait - #9943
Conversation
…CLI, Apollo OAuth people search, and one short SailPoint throttle wait
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
There was a problem hiding this comment.
1 issue found across 19 files
Confidence score: 4/5
people_search.tscan report page 0 when the request used page 1. Use(params?.page || 1)as the fallback.
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. When an issue isn't valid or won't be fixed in this PR, reply in its thread with the reason and then resolve the thread. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="apps/sim/tools/apollo/people_search.ts">
<violation number="1" location="apps/sim/tools/apollo/people_search.ts:199">
P3: `transformResponse` can report page 0 even though the request sent page 1. Use `(params?.page || 1)` for the fallback.</violation>
</file>
Reply with feedback, questions, or to request a fix.
Turn on auto-fix | Re-trigger cubic
[Medium impact] The PR appears safe to merge; no new actionable issues were found. SummaryAdds four credential fixes:
The latest changes preserve Splunk password spaces and make Apollo’s reported page match the page requested. The earlier, unnumbered Splunk password finding is fully fixed. No new actionable issues were found. Reviews (2) · Last reviewed commit: "fix(credentials): keep a Splunk password..." · Reviewed by Greptile |
…port page 1 when Apollo is asked for page 0
|
@cubic-dev-ai review this PR |
@waleedlatif1 I have started the AI code review. It will take a few minutes to complete. |
Summary
Fixes for the four issues a release audit of main..staging confirmed. Each was checked against the code and the vendor's docs before it was fixed. Existing credentials, blocks and CLI calls behave as before.
splunkblock is sunset, andsplunk_v2only took a token, but token sign-in is off by default on Splunk Enterprise.authentication/current-context, and the credential resolves withauthStyle: 'basic'. The executor setsauthStyle; a caller can't.sim credentials create snowflake-service-accountrefused a key-pairserviceAccountJsonon deployments that can't sign keyless tokens, and refused a raw key-pair object everywhere.serviceAccountJsonnow stands in for the required fields whenever the catalog offers it as an optional field, which matches what the server accepts.authTypeis wrapped for those providers./mixed_people/api_search.mixed_people_api_search) covers that path, not the legacy/mixed_people/search.Type of Change
Testing
people_search, and SailPoint minter.tscforpackages/sim-cli.bun run lintandgenerate:tool-metadataran. These pass:check:tool-metadata,check:test-patterns,check:comment-hygiene,check:explicit-any,check:boundaries,check:client-boundary,check:utils,check:file-names,check:api-validation:strict,check:cli-docs,check:docs,check:tool-registry-boundary --checkandcheck-block-registry.ts origin/staging.Checklist
test-auditauthoring gate)