Skip to content

Fix Windows auth URL opener#16

Merged
MaheshtheDev merged 4 commits into
mainfrom
codex/windows-auth-opener
Jun 7, 2026
Merged

Fix Windows auth URL opener#16
MaheshtheDev merged 4 commits into
mainfrom
codex/windows-auth-opener

Conversation

@ishaanxgupta
Copy link
Copy Markdown
Contributor

@ishaanxgupta ishaanxgupta commented Jun 3, 2026

What changed

  • Replaced the Codex login flow's Windows explorer.exe URL launch path with a guarded shared openUrl helper.
  • The Windows path now uses rundll32.exe url.dll,FileProtocolHandler first, with a cmd /c start fallback.
  • Added a regression test that verifies the bundled login script contains the safe opener and does not include explorer.exe.
  • Fixed Windows test isolation by using fileURLToPath for spawned bundle paths and setting USERPROFILE alongside HOME.
  • add /status and /logout

Why

On Windows, launching the long OAuth URL through explorer.exe can be interpreted as a filesystem target instead of a browser URL, causing /supermemory-login to redirect into an internal/local directory rather than the Supermemory authentication flow. This mirrors the issue previously fixed in the Amp plugin.

@ishaanxgupta ishaanxgupta changed the title [codex] Fix Windows auth URL opener Fix Windows auth URL opener Jun 3, 2026
@ishaanxgupta ishaanxgupta marked this pull request as ready for review June 3, 2026 11:58
Comment thread src/services/client.ts
@MaheshtheDev MaheshtheDev merged commit d68ea27 into main Jun 7, 2026
2 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