Skip to content

Add Open Cosmos integration flow#64

Open
DattSpark wants to merge 2 commits into
mainfrom
feature/open-cosmos-support
Open

Add Open Cosmos integration flow#64
DattSpark wants to merge 2 commits into
mainfrom
feature/open-cosmos-support

Conversation

@DattSpark

Copy link
Copy Markdown
Contributor

Summary

Adds Open Cosmos account connection support to the Linked Accounts page using the OAuth 2.0 Authorization Code flow with PKCE (Proof Key for Code Exchange).

Changes

  • Add Open Cosmos connect and disconnect controls.
  • Exchange authorization codes and refresh expired access tokens.
  • Send OAuth credentials to workspace-services for per-user, per-workspace storage.
  • Keep tokens out of browser persistent storage.
  • Restore the selected workspace menu page after the OAuth redirect. (src\components\WorkspaceMenu\WorkspaceMenu.tsx)
  • Prevent duplicate authorization-code exchanges in React StrictMode.
  • Add the required Open Cosmos environment configuration (new env vars).

Validation

image image image

@DattSpark DattSpark requested review from GeoSpark and geodowd June 15, 2026 04:52

@GeoSpark GeoSpark 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.

LGTM

@GeoSpark

Copy link
Copy Markdown

Are the Trivy scan failures worth fixing? It seems it just needs some packages updating.

@DattSpark

Copy link
Copy Markdown
Contributor Author

Are the Trivy scan failures worth fixing? It seems it just needs some packages updating.

Yes, It's worth resolving them, but I think it should be a separate pull request. One that focuses solely on upgrading those libraries and testing that everything is working correctly afterward.

@DattSpark DattSpark requested a review from GeoSpark June 15, 2026 18:09
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