Skip to content

Performance: getTraefikConfig anySitesOnline out of loop / isDomainCoveredByWildcard bypass logic on matching domains.#3191

Open
shleeable wants to merge 2 commits into
fosrl:mainfrom
shleeable:patch-2
Open

Performance: getTraefikConfig anySitesOnline out of loop / isDomainCoveredByWildcard bypass logic on matching domains.#3191
shleeable wants to merge 2 commits into
fosrl:mainfrom
shleeable:patch-2

Conversation

@shleeable

@shleeable shleeable commented May 30, 2026

Copy link
Copy Markdown
Contributor

Community Contribution License Agreement

By creating this pull request, I grant the project maintainers an unlimited,
perpetual license to use, modify, and redistribute these contributions under any terms they
choose, including both the AGPLv3 and the Fossorial Commercial license terms. I
represent that I have the right to grant this license for all contributed content.

Description

  • server/lib/traefik/getTraefikConfig.ts - move the anySitesOnline const out of the loop. Fixes redundant iterations over the target sites list.
  • server/lib/traefik/TraefikConfigManager.ts - minor check that just returns true for non-complex wildcard certs.

How to test?

@shleeable shleeable changed the title Performance Tweaks Performance: getTraefikConfig anySitesOnline out of loop / isDomainCoveredByWildcard bypass logic on matching domains. May 30, 2026
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.

1 participant