Skip to content

feat: add Earth Engine OAuth flow for packaged desktop app - #31

Merged
giswqs merged 1 commit into
mainfrom
feat/earthengine-tauri-oauth
May 29, 2026
Merged

feat: add Earth Engine OAuth flow for packaged desktop app#31
giswqs merged 1 commit into
mainfrom
feat/earthengine-tauri-oauth

Conversation

@giswqs

@giswqs giswqs commented May 29, 2026

Copy link
Copy Markdown
Member

Summary

  • Add a localhost OAuth helper server in the Tauri backend that serves a Google sign-in page and relays the Earth Engine access token back to the GeoAgent plugin, fixing sign-in in packaged builds where the Google Identity token client cannot run from tauri:///file:// origins.
  • Detect the production origin in the plugin and route authentication through the Tauri helper, while keeping the existing in-browser flow for development.
  • Propagate nested OAuth popup window handling and improve error message extraction for sign-in failures.

Test plan

  • npm run build passes (pre-commit npm-build hook)
  • Dev server sign-in still uses the in-browser Google Identity flow
  • Packaged desktop build completes Earth Engine sign-in via the localhost helper and loads Earth Engine tiles

The Google Identity token client cannot run from the tauri:// or
file:// origins used in production builds, so sign-in only worked in
the dev server. This adds a localhost OAuth helper server in Tauri that
serves a Google sign-in page and relays the access token back to the
plugin, with the in-browser flow preserved for development.
Copilot AI review requested due to automatic review settings May 29, 2026 19:23
@github-actions

Copy link
Copy Markdown
Contributor

Netlify preview: https://pr-31--opengeos.netlify.app

@giswqs
giswqs merged commit 19459e1 into main May 29, 2026
4 of 5 checks passed
@giswqs
giswqs deleted the feat/earthengine-tauri-oauth branch May 29, 2026 19:26

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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

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