Skip to content

[comp] Production Deploy#3231

Merged
tofikwest merged 3 commits into
releasefrom
main
Jun 22, 2026
Merged

[comp] Production Deploy#3231
tofikwest merged 3 commits into
releasefrom
main

Conversation

@github-actions

@github-actions github-actions Bot commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

This is an automated pull request to release the candidate branch into production, which will trigger a deployment.
It was created by the [Production PR] action.


Summary by cubic

Persisted api_domain from multi-DC OAuth providers (e.g., Zoho) so tokens target the correct regional host, fixing INVALID_TOKEN errors for non-US accounts. Also preserves the prior value on refresh when the provider omits it.

  • Bug Fixes
    • Store api_domain in the credential vault and keep the previous value if a refresh omits it; single-DC providers remain unchanged.
    • Expose api_domain in OAuth controller types to pass through callback responses.
    • Add tests for capture, preservation, omission, and decrypt-failure fallback.

Written for commit be592f7. Summary will update on new commits.

Review in cubic

github-actions Bot and others added 3 commits June 22, 2026 17:39
…it the right region

Zoho (and other multi-data-center OAuth providers) return a data-center-
specific API host as `api_domain` in their token and refresh responses, and
the access token only works against that host. We dropped this field, so
checks fell back to the default US host (www.zohoapis.com) and Zoho rejected
the token with INVALID_TOKEN for any non-US (EU/IN/AU/JP/CN) account.

Capture and persist `api_domain` from the token-exchange and refresh
responses (preserving a previously captured value when a refresh response
omits it). The dynamic-check runtime already reads
`ctx.credentials.api_domain` with a www.zohoapis.com fallback, so no check
change is needed.

Strictly additive and guarded: `api_domain` is a Zoho-only field, so every
other OAuth provider (GitHub, Google, etc.) stores nothing new and behaves
byte-for-byte as before. Stored in plaintext alongside scope/token_type so the
runtime can route requests to the correct regional host.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AU5798EG3PQdRuYSPJgXmy
…omain

fix(integrations): persist Zoho multi-DC api_domain so OAuth tokens hit the right region
@vercel

vercel Bot commented Jun 22, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
comp-framework-editor (staging) Building Building Preview Jun 22, 2026 7:14pm
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
app (staging) Skipped Skipped Jun 22, 2026 7:14pm
portal (staging) Skipped Skipped Jun 22, 2026 7:14pm

Request Review

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No issues found across 3 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Re-trigger cubic

@tofikwest tofikwest merged commit d655227 into release Jun 22, 2026
12 checks passed
@claudfuen

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 3.87.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants