Skip to content

fix(inigo): use TLS NATS (TNZ-1234) - #1221

Merged
kart2bc merged 1 commit into
developfrom
inigo-nats-tls
Oct 21, 2026
Merged

kart2bc merged 1 commit into
developfrom
inigo-nats-tls

Conversation

@kart2bc

@kart2bc kart2bc commented Oct 20, 2026

Copy link
Copy Markdown
Contributor

Upstream routing-release commit 5e0558f removed non-TLS NATS support, so gorouter now fails to load its config when nats.ca_certs is empty. Inigo's componentMaker generated a gorouter config without TLS NATS.

  • Add NATS CA, server and client fixture certs (fixtures/certs/nats), generated by a new block in regenerate-certs.sh, and start nats-server with --tls and --tlsverify.
  • Give gorouter the NATS CA, client cert and client key as inline PEM, and set nats.tls_enabled so forks that still treat TLS as opt-in connect too.
  • Enable NATS TLS in the route-emitter config.

ai-assisted=yes
TNZ-1234

Summary

Backward Compatibility

Breaking Change? No

Gorouter no longer supports non-TLS NATS and fails to load its config
when nats.ca_certs is empty. Inigo's componentMaker generated a gorouter
config without TLS NATS.

- Add NATS CA, server and client fixture certs (fixtures/certs/nats),
  generated by a new block in regenerate-certs.sh, and start nats-server
  with --tls and --tlsverify.
- Give gorouter the NATS CA, client cert and client key as inline PEM, and
  set nats.tls_enabled so forks that still treat TLS as opt-in connect too.
- Enable NATS TLS in the route-emitter config.

ai-assisted=yes
TNZ-1234
@github-project-automation github-project-automation Bot moved this from Inbox to Pending Merge | Prioritized in Application Runtime Platform Working Group Oct 21, 2026
@kart2bc
kart2bc merged commit 7ed9938 into develop Oct 21, 2026
10 checks passed
@github-project-automation github-project-automation Bot moved this from Pending Merge | Prioritized to Done in Application Runtime Platform Working Group Oct 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

3 participants