Skip to content

document ALPN loadbalancer issues; use manager TLSConfig for autocert#221

Merged
Roasbeef merged 2 commits intolightninglabs:masterfrom
starius:2603-alpn-fix-2
Mar 24, 2026
Merged

document ALPN loadbalancer issues; use manager TLSConfig for autocert#221
Roasbeef merged 2 commits intolightninglabs:masterfrom
starius:2603-alpn-fix-2

Conversation

@starius
Copy link
Copy Markdown
Contributor

@starius starius commented Mar 20, 2026

  1. Document loadbalancer ALPN requirements for gRPC deployments. TLS-terminating load balancers must advertise h2 over ALPN, or clients can fail with "missing selected ALPN property". Call out AWS NLB default ALPN policy and recommended options. Note that TCP passthrough keeps ALPN negotiation at aperture.

  2. Use autocert.Manager.TLSConfig() instead of assembling GetCertificate TLS configs manually. This keeps autocert's ALPN and ACME defaultswhile preserving Aperture's cipher suite and minimum TLS version
    settings.

Replaces #213

starius added 2 commits March 19, 2026 23:43
Use autocert.Manager.TLSConfig() instead of assembling GetCertificate
TLS configs manually. This keeps autocert's ALPN and ACME defaults
while preserving Aperture's cipher suite and minimum TLS version
settings.
Document loadbalancer ALPN requirements for gRPC deployments.
TLS-terminating load balancers must advertise h2 over ALPN, or
clients can fail with "missing selected ALPN property".

Call out AWS NLB default ALPN policy and recommended options.
Note that TCP passthrough keeps ALPN negotiation at aperture.
@starius starius mentioned this pull request Mar 20, 2026
1 task
Copy link
Copy Markdown
Member

@Roasbeef Roasbeef left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🥬

@Roasbeef Roasbeef merged commit cf0f4fc into lightninglabs:master Mar 24, 2026
6 of 7 checks passed
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