Redirect plain HTTP to HTTPS on direct TLS

オープン
#440 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

メンテナーはふだん 1 日以内に返信

まだ誰も着手していません。

評価

難易度
4/5
見積もり時間
3〜5日
初心者へのやさしさ
35/100
issue の種類
機能追加
明瞭さ
明確に書かれている
活発さ
活発
技術スタック
go
領域
backend

調査の方向性

Read internal/server/http.go and the TLS guide at site/src/content/docs/docs/guides/tls.mdx; first trace how direct TLS and FANOUT_PUBLIC_URL are configured. The issue is explicitly deferred until users request it, so confirm that demand exists before starting. Done means an opt-in port 80 listener redirects only to the configured HTTPS origin, startup rejects the setting without TLS, and the configuration reference and guide are updated.

索引モデルが issue の本文から書いたものです。

説明

[!NOTE]
Deferred. Reverse proxies and CDNs already do this redirect, and a built-in one needs a second listener, which departs from the one-port design. Pick this up if people running direct TLS ask for it.

What

With direct TLS, the single listener serves only HTTPS, so someone who types a bare hostname into a browser gets a connection failure. An optional listener on port 80 could answer with a redirect to the HTTPS origin.

Acceptance criteria

  • Off by default; the one-port deployment is unchanged unless the setting is present
  • The listener serves only redirects, with no API, UI, MCP, or OTLP on it
  • The redirect target comes from configuration (FANOUT_PUBLIC_URL), never from the request Host header
  • Startup fails with an error naming the setting when TLS is not enabled
  • Configuration reference and TLS guide updated

References

  • internal/server/http.go
  • site/src/content/docs/docs/guides/tls.mdx

One pull request. Use the latest stable version of any dependency added, verified against its registry and pinned exactly.

主要言語
Go
スター
11
フォーク
0
平均マージ
7時間 5分
マージ済み PR(30日)
112

環境構築

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

labstack/fanout のほかの issue

labstack/fanout の issue をすべて見る

似ている issue

Go の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。