agentic-community / agentic-community/mcp-gateway-registry
Expose alb.ingress.kubernetes.io/ssl-policy as a configurable value
- 主要言語
- Python
- スター
- 912
- フォーク
- 234
- 平均マージ
- 1日 11時間
- マージ済み PR(30日)
- 62
説明
charts/registry/templates/ingress.yaml hardcodes several ALB annotations (listen-ports, ssl-redirect, target-type, etc.) and exposes inboundCidrs as a values-driven extensibility point, but there's no way to set alb.ingress.kubernetes.io/ssl-policy. Without it, the AWS Load Balancer Controller falls back to its own legacy default (ELBSecurityPolicy-2016-08), which can cause TLS handshake failures with some client-side security proxies/brokers expecting a more modern policy (e.g. ELBSecurityPolicy-TLS-1-2-Ext-2018-06).
Workaround: manually annotate the live Ingress object outside of Helm.
Suggested fix: add an optional global.ingress.sslPolicy value, following the same pattern as inboundCidrs.
コントリビューションガイド
調査の方向性
Check charts/registry/templates/ingress.yaml, where the ALB annotations are defined, and confirm that alb.ingress.kubernetes.io/ssl-policy is not currently configurable. Then inspect the chart values file (such as charts/registry/values.yaml) for the existing inboundCidrs/global.ingress pattern and add a parallel global.ingress.sslPolicy option. Run helm template or helm lint to render the chart and verify the ingress manifest includes the ssl-policy annotation when the value is set. Done means Helm users can configure SSL policy via values instead of manual ingress edits.
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- aws, kubernetes
- 領域
- infrastructure
- issue の種類
- 機能追加
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 活発さ
- 活発
- 明瞭さ
- 明確に書かれている
- 初心者へのやさしさ
- 78/100