Skip to content

caddytls: Consolidate empty APs more smartly#7567

Merged
mholt merged 2 commits intomasterfrom
fix-ap-consolidation
Mar 26, 2026
Merged

caddytls: Consolidate empty APs more smartly#7567
mholt merged 2 commits intomasterfrom
fix-ap-consolidation

Conversation

@mholt
Copy link
Copy Markdown
Member

@mholt mholt commented Mar 13, 2026

This cleans up automation policy conslidation in some cases

Should fix #7559.

Assistance Disclosure

No AI was used.

@mohammed90 mohammed90 changed the title caddytls: Consoldate empty APs more smartly caddytls: Consolidate empty APs more smartly Mar 25, 2026
@mholt
Copy link
Copy Markdown
Member Author

mholt commented Mar 25, 2026

@filiabel How would you feel about trying this out

@filiabel
Copy link
Copy Markdown

@mholt Seems to be working with the little cert-server runtime test 👍

=== Starting cert-server ===
=== Building caddy via xcaddy from branch fix-ap-consolidation ===
=== Running caddy (fix-ap-consolidation)  ===
  wiki.example.com  → 200
  other.example.com → 200
  cert-server was called for:
    wiki.example.com
    other.example.com
  caddy tls handshake log:
    wiki.example.com → using externally-managed certificate
    other.example.com → using externally-managed certificate

@mholt
Copy link
Copy Markdown
Member Author

mholt commented Mar 26, 2026

Great! Thank you for verifying!

@mholt mholt merged commit acf8d6a into master Mar 26, 2026
24 of 28 checks passed
@mholt mholt deleted the fix-ap-consolidation branch March 26, 2026 20:41
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.

Regression for wildcard tls get_certificate http in v2.11

2 participants