Skip to content

Renew expired TLS test certificates - #972

Merged
wallyqs merged 1 commit into
mainfrom
renew-tls-test-certs
Jun 4, 2026
Merged

Renew expired TLS test certificates#972
wallyqs merged 1 commit into
mainfrom
renew-tls-test-certs

Conversation

@caspervonb

Copy link
Copy Markdown
Collaborator

The server and client leaf certificates in nats-core/tests/certs and nats/tests/certs expired on 2026-06-03, so every TLS and WebSocket test fails with certificate verify failed: certificate has expired.

Re-signed both leaf certs with the existing CA and private keys, preserving the subjects, SANs (DNS:localhost, IP:127.0.0.1), and key usage. Validity now runs to 2035 (within the CA's window) instead of one year, so this stops recurring annually.

Supersedes #971, which was accidentally merged into the #970 branch instead of main.

The server and client leaf certs in nats-core and nats expired on
2026-06-03, breaking every TLS and WebSocket test. Re-sign both with
the existing CA and keys, preserving subjects, SANs, and key usage, and
extend validity to 2035 (within the CA window) so this stops recurring
annually.
@caspervonb caspervonb added nats-core nats-legacy Legacy asyncio NATS client labels Jun 4, 2026
@synadia-claude-reviewer

synadia-claude-reviewer Bot commented Jun 4, 2026

Copy link
Copy Markdown

Claude finished @caspervonb's task in 2m 1s —— View job


PR Review: Renew expired TLS test certificates

  • Gather context and understand the PR
  • Examine the PR diff and certificate files
  • Verify certificate properties (expiry, SANs, subject, CA chain)
  • Post review findings

This PR is a clean, minimal certificate renewal. Here's what I verified from the diff and the unchanged CA cert:

Property Old New
Server cert validity 2025-06-03 → 2026-06-03 (expired) 2026-06-04 → 2035-03-09
Client cert validity 2025-06-03 → 2026-06-03 (expired) 2026-06-04 → 2035-03-09
CA validity 2025-06-03 → 2035-06-01
SANs (DNS:localhost, IP:127.0.0.1) ✓ preserved
Subject (CN=localhost / CN=Test Client) ✓ preserved
AKID → CA SKID chain ✓ preserved
Private keys changed No (only certs re-signed)
nats/ and nats-core/ certs identical ✓ preserved

The new leaf certs expire 2035-03-09, ~84 days before the CA expires (2035-06-01) — well within the CA's validity window. No issues found.

@caspervonb
caspervonb marked this pull request as ready for review June 4, 2026 12:01

@Jarema Jarema left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@wallyqs
wallyqs merged commit 318c818 into main Jun 4, 2026
50 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

nats-core nats-legacy Legacy asyncio NATS client

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants