Skip to content

fix(cms-auth): GITHUB_CLIENT_ID を wrangler.toml で管理し CMS ログインを復旧する - #323

Merged
azumag merged 1 commit into
mainfrom
fix/cms-auth-client-id
Oct 22, 2026
Merged

azumag merged 1 commit into
mainfrom
fix/cms-auth-client-id

Conversation

@azumag

@azumag azumag commented Oct 22, 2026

Copy link
Copy Markdown
Owner

背景

Sveltia CMS に新しくログインしようとすると(シークレットウィンドウなど)、「OAuth アプリのクライアント ID またはシークレットが設定されていません」と表示されて失敗します。

  • 原因は、本番の sveltia-cms-auth Worker に GITHUB_CLIENT_SECRET しか残っていないことです。fix(cms-auth): ALLOWED_DOMAINS を wrangler.toml で管理し、配備で消えないようにする #322 の後に /auth が MISCONFIGURED_CLIENT を返すことを実測し、利用者もシークレットウィンドウで同じエラーを確認しています。
  • Cloudflare に残っている最古の version(10/5)の時点で、すでに Client ID はありません。
  • ログイン済みのブラウザは保存済みのトークンで動くため、普段使いでは気づきにくい状態でした。

変更

  • workers/sveltia-cms-auth/wrangler.toml の [vars] に GITHUB_CLIENT_ID を追加しました。
    • Client ID は認可 URL に載る公開値なので、ALLOWED_DOMAINS と同じく repo で管理し、配備のたびに必ず設定されるようにします。
  • Client Secret はこれまでどおり secret です(既存の GITHUB_CLIENT_SECRET が残っていることは確認済みです)。
  • README の手順を、この運用に合わせて更新しました。

検証

  • wrangler deploy --dry-run --config workers/sveltia-cms-auth/wrangler.toml(wrangler 4.151.3)で、env.ALLOWED_DOMAINS と env.GITHUB_CLIENT_ID が binding に出ることを確認しました。
  • マージ後に実施すること:
    • deploy log で binding を確認する
    • /auth?provider=github&site_id=www.bluemoon.works が GitHub の認可 URL に転送されることを確認する
    • シークレットウィンドウからのログインを確認する

🤖 Generated with Claude Code

https://claude.ai/code/session_01WthszjoYsBn39cjTaRXQ5M

本番の sveltia-cms-auth Worker には GITHUB_CLIENT_SECRET しか残っておらず、
新規ログインが MISCONFIGURED_CLIENT になっていた(ログイン済みブラウザは保存済み
トークンで動くため気づきにくい)。Client ID は認可 URL に載る公開値なので、
ALLOWED_DOMAINS と同じく [vars] で管理して配備ごとに必ず設定する。

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01WthszjoYsBn39cjTaRXQ5M
@github-actions

Copy link
Copy Markdown

🔍 Cloudflare Workers preview: https://bluemoon-pr-323.tsubasa-azumagakito.workers.dev
Worker: bluemoon-pr-323

(このコメントは新しいpushのたびに更新されます)

@azumag
azumag merged commit ce4233a into main Oct 22, 2026
3 checks passed
@azumag
azumag deleted the fix/cms-auth-client-id branch October 22, 2026 05:46
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