Skip to content

fix(setup-ci-app): serve form via localhost, open default browser#286

Merged
jdfalk merged 1 commit into
mainfrom
fix/setup-ci-app-localhost-browser
May 17, 2026
Merged

fix(setup-ci-app): serve form via localhost, open default browser#286
jdfalk merged 1 commit into
mainfrom
fix/setup-ci-app-localhost-browser

Conversation

@jdfalk
Copy link
Copy Markdown
Owner

@jdfalk jdfalk commented May 17, 2026

Summary

  • Fixes 500 error: browsers block file://https:// cross-origin POSTs (GitHub CSRF protection)
  • Fixes "not logged in" issue: removed incognito logic, opens in default browser
  • Python server now handles 2 requests: GET / serves the manifest form, GET /callback receives the code
  • Adds issues: "write" to the app manifest (required for burndown-tasks TODO sync)

Root cause

Old script wrote HTML to a temp file and opened file:///tmp/.../go.html. Browsers treat file:// as opaque origin and block POSTs to https://github.com — GitHub returned 500. Script also opened incognito so user wasn't logged in.

Test plan

  • Run ./scripts/setup-ci-app.sh — browser opens http://localhost:8765/ (not file://)
  • Form auto-submits to GitHub; user clicks "Create GitHub App"
  • GitHub redirects back; terminal shows "✓ Callback received"

🤖 Generated with Claude Code

File:// URLs block cross-origin POSTs to github.com (CSRF); incognito
mode means the user isn't logged in. Python server now handles both the
root / (HTML form) and /callback (code exchange), then the script opens
http://localhost:PORT/ in the default browser instead of incognito.

Also adds issues:write to the app manifest so burndown-tasks sync works.

Co-Authored-By: Claude Sonnet 4.6 <[email protected]>
@jdfalk jdfalk merged commit 41c69d5 into main May 17, 2026
5 of 11 checks passed
@jdfalk jdfalk deleted the fix/setup-ci-app-localhost-browser branch May 17, 2026 23:34
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.

1 participant