Skip to content

公開されるworkers.devホスト名を無効化する - #11

Merged
rin2yh merged 3 commits into
mainfrom
claude/domain-visibility-settings-59y5r1
Jul 26, 2026
Merged

公開されるworkers.devホスト名を無効化する#11
rin2yh merged 3 commits into
mainfrom
claude/domain-visibility-settings-59y5r1

Conversation

@rin2yh

@rin2yh rin2yh commented Jul 26, 2026

Copy link
Copy Markdown
Contributor

概要

  • wrangler.jsonc で公開ホスト名を無効化し、この Worker への公開 HTTP 入口をなくした
    • workers_dev: falseemail-notifier.<subdomain>.workers.dev
    • preview_urls: false — バージョンごとの <version>-email-notifier.<subdomain>.workers.dev
  • 併せて send_metrics: false で Wrangler のテレメトリ送信を止めた

設計判断

  • 公開ホスト名は両方オフにする
    • この Worker は email() のみで fetch ハンドラを持たないため、公開 URL は何も返さない。到達できても「デプロイが存在する」ことを外部に示すだけの公開面で、得るものがない
  • observability.enabled は有効のまま残す
    • Email Worker は手元から叩けないため、email() の失敗を追える唯一の経路。出力先はアカウント内のログで公開されず、コードがログに出すのはエラーのみでメール本文は含まれない
  • upload_source_maps / logpush は明示指定せず既定(オフ)のまま
    • 既定で無効なので、設定を増やす意味がない

レビュー観点

内部品質

  • 3 つの設定それぞれに、なぜオフなのか(および observability がなぜオンのままか)をコメントで残した。既定値に戻す変更が意図せず入るのを防ぐ狙い
  • README の Deployment に「Email Routing 経由でのみ到達可能」である旨を追記した

外部品質

  • wrangler deploy --dry-run が警告なく完了することを確認(4.110.0、CI が使うバージョンと同じ)
    • routes 未設定 + workers_dev: false でもアップロードは成立する。メールのトリガーは Email Routing 側のバインディングで、Worker の HTTP ルートとは独立しているため
  • bun run typecheck / lint / test(18 tests)がいずれも成功。挙動を変える変更ではないためテストの追加はしていない

補足

  • workers_devroutes 未設定時の既定が true。つまり今回の明示指定が入るまでは workers.dev ホスト名が有効な状態でデプロイされていた
  • 反映は main マージ後のデプロイ実行時。既存の workers.dev ルートはそのデプロイで削除される
  • Cloudflare ダッシュボード側にも Worker ごとの有効/無効トグルがあるが、次回以降は本ファイルの設定が優先される

Generated by Claude Code

claude and others added 3 commits July 26, 2026 14:59
Email Routing からのみ起動される Worker で `fetch` ハンドラを持たないため、
`email-notifier.<subdomain>.workers.dev` と各バージョンの Preview URL は
用途のない公開面になっていた。`routes` 未設定時 `workers_dev` は true が
既定なので、明示的に両方を無効化する。

併せて Wrangler のテレメトリ送信も送信しない設定にした。observability は
email() の失敗を追う唯一の手段であり、記録先はアカウント内なので有効のまま
維持する。
@rin2yh
rin2yh marked this pull request as ready for review July 26, 2026 15:04
@rin2yh
rin2yh merged commit be94f96 into main Jul 26, 2026
2 checks passed
@rin2yh
rin2yh deleted the claude/domain-visibility-settings-59y5r1 branch July 26, 2026 15:04
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