Skip to content

fix(ENG-12551): update connection attempts URL to /hub/ path#137

Merged
adefreitas merged 1 commit into
StackOneHQ:mainfrom
adefreitas:ENG-12551/update-connection-attempts-url
Apr 16, 2026
Merged

fix(ENG-12551): update connection attempts URL to /hub/ path#137
adefreitas merged 1 commit into
StackOneHQ:mainfrom
adefreitas:ENG-12551/update-connection-attempts-url

Conversation

@adefreitas

@adefreitas adefreitas commented Apr 16, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Update connection attempt URLs from /connect_sessions/connection_attempts to /hub/connection_attempts

Context

CloudFront's Lambda@Edge handles CORS for /hub/ paths. Moving the endpoints there avoids CORS issues without infrastructure changes. Pairs with StackOneHQ/unified-cloud-api#8037.

Test plan

  • Deploy API change first, then this
  • Verify polling works end-to-end from Hub embed

🤖 Generated with Claude Code


Summary by cubic

Switch connection attempt API calls from /connect_sessions/connection_attempts to /hub/connection_attempts to use CloudFront CORS. Enables reliable server-side status polling for the Hub embed (ENG-12551).

  • Migration
    • Deploy backend support for /hub/connection_attempts before this change.
    • Verify polling works end-to-end from the Hub embed.

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

Move from /connect_sessions/connection_attempts to
/hub/connection_attempts to leverage CloudFront's existing
CORS handling for /hub/ paths.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings April 16, 2026 20:00

@claude claude Bot 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.

Claude Code Review

This pull request is from a fork — automated review is disabled. A repository maintainer can comment @claude review to run a one-time review.

Copilot AI 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.

Pull request overview

Updates the Integration Picker’s connection-attempt API calls to use the /hub/ path so requests benefit from CloudFront Lambda@Edge CORS handling when embedded.

Changes:

  • Switched createConnectionAttempt, pollConnectionAttempt, and cancelConnectionAttempt endpoints from /connect_sessions/connection_attempts to /hub/connection_attempts.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@cubic-dev-ai cubic-dev-ai Bot 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.

No issues found across 1 file

Requires human review: This change modifies API endpoints which requires backend coordination and could break connection attempts if deployed out of sync. API changes require human review.

@adefreitas adefreitas merged commit 059363d into StackOneHQ:main Apr 16, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants