agentic-community / agentic-community/mcp-gateway-registry

Expose alb.ingress.kubernetes.io/ssl-policy as a configurable value

未关闭 适合新手
#1,661 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
主要语言
Python
星标
912
派生
234
平均合并
1 天 11 小时
30 天内合并 PR
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

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。