Skip to content

feat: sign in with GitHub - #41

Merged
tknkaa merged 3 commits into
mainfrom
feat/github-login
Oct 2, 2026
Merged

tknkaa merged 3 commits into
mainfrom
feat/github-login

Conversation

@tknkaa

@tknkaa tknkaa commented Oct 2, 2026

Copy link
Copy Markdown
Collaborator

Refs #2

ログインに GitHub OAuth を追加する (Google ではなく GitHub にした)。キーを入れるまでは、ボタンは出ない。 GitHub の OAuth App の作成と、本番へのキーの設定は、まだ。

仕組み

  • GITHUB_CLIENT_ID と GITHUB_CLIENT_SECRET の両方がある環境でだけ有効。環境の名前ではなく、設定の有無で分ける。ない環境 (ローカルで書いていないとき、プレビュー) では、ボタンが出ない
  • ログイン画面は GET /api/config ({ github: true | false }、秘密は返さない) で、ボタンを出すかを決める
  • 同じメールアドレスのメール/パスワードのアカウントが、すでにあれば、ひとつにまとめる (GitHub が確認済みと伝えたメールアドレスだけ)
  • 共有の招待は、GitHub でアカウントを作ったときにも反映される (サインアップのフックを共有している)

テスト

  • GitHub の認可の URL が、正しい client_id、redirect_uri (<URL>/api/auth/callback/github)、user:email の権限で作られる (better-auth のハンドラを通して確認。GitHub には接続しない)
  • キーがないとき、GitHub でのログインは断られる
  • キーの有無の判定は、純粋な関数にしてテスト
  • E2E: キーのないテストのサーバーで、ボタンが出ない

注意

  • GitHub の OAuth App は、戻り先の URL を 1 つしか登録できない。本番とローカルで別のアプリを作る (docs/web-app.md に手順)
  • ブランチごとのプレビューでは使えない (戻り先に、複数の URL もワイルドカードも登録できない)。プレビューにはキーを入れない
  • 実際に GitHub でログインできることの確認は、OAuth App を作って、キーを入れてから行う

docs を更新 (Web アプリ仕様、デプロイ、README、.dev.vars.example)

@tknkaa tknkaa mentioned this pull request Oct 2, 2026
3 of 6 tasks
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Oct 2, 2026 •

Copy link
Copy Markdown

🚀 Deploying Preview to Cloudflare 🚀

Preview URL: https://feat-github-login-kanban-web.ut-code.workers.dev (commit f73b3ac)

This URL reflects your latest Preview deployment

Preview Deployments by commit

Status Deployment URL Commit Updated (UTC) See this deployment's details
  • Build: Success ✅
  • Deployment: Success ✅

View logs ↗
https://18d75d83-kanban-web.ut-code.workers.dev f73b3ac 2026-10-02T14:14:09.487Z Visit the dashboard ↗
  • Build: Success ✅
  • Deployment: Success ✅

View logs ↗
https://300858c8-kanban-web.ut-code.workers.dev ded0f86 2026-10-02T14:01:28.709Z Visit the dashboard ↗
  • Build: Success ✅
  • Deployment: Success ✅

View logs ↗
https://83fe76fe-kanban-web.ut-code.workers.dev 0ed8ab9 2026-10-02T13:51:42.109Z Visit the dashboard ↗

@tknkaa
tknkaa merged commit b22414a into main Oct 2, 2026
3 checks passed
@tknkaa
tknkaa deleted the feat/github-login branch October 2, 2026 14:14
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