don't check for the len of hosts when evaluating the TLSOnDemandUrl

basecamp · Nov 30, 2024 · 2e0ef39 · 2e0ef39
1 parent f91ed90
commit 2e0ef39
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/internal/server/service.go b/internal/server/service.go
@@ -337,9 +337,9 @@ func (s *Service) createCertManager(hosts []string, options ServiceOptions) (Cer
 }
 
 func (s *Service) createAutoCertHostPolicy(hosts []string, options ServiceOptions) (autocert.HostPolicy, error) {
-	onDemandTls := len(hosts) == 0 && options.TLSOnDemandUrl != ""
+	slog.Info("createAutoCertHostPolicy called", options.TLSOnDemandUrl, len(hosts), "🚨", "ok")
 
-	if !onDemandTls {
+	if options.TLSOnDemandUrl == "" {
 		return autocert.HostWhitelist(hosts...), nil
 	}
 
@@ -350,7 +350,11 @@ func (s *Service) createAutoCertHostPolicy(hosts []string, options ServiceOption
 		return nil, err
 	}
 
+	slog.Info("Will use the tls_on_demand_url URL")
+
 	return func(ctx context.Context, host string) error {
+		slog.Info("Get a certificate for", host, "🤞")
+
 		resp, err := http.Get(fmt.Sprintf("%s?host=%s", options.TLSOnDemandUrl, url.QueryEscape(host)))
 
 		if err != nil {