Skip to content

Commit 681f00c

Browse files
authored
Merge pull request #70 from Dstack-TEE/correct-propagation-opt
ingress: Correct propagation option for other providers
2 parents da84773 + f726677 commit 681f00c

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

custom-domain/dstack-ingress/scripts/certman.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -291,7 +291,10 @@ def _build_certbot_command(self, action: str, domain: str, email: str) -> List[s
291291
if os.environ.get("CERTBOT_STAGING", "false") == "true":
292292
base_cmd.extend(["--staging"])
293293

294-
base_cmd.extend(["--dns-cloudflare-propagation-seconds=120"])
294+
if getattr(self.provider, 'CERTBOT_PROPAGATION_SECONDS'):
295+
propagation_seconds = self.provider.CERTBOT_PROPAGATION_SECONDS
296+
propagation_param = f"--dns-{self.provider_type}-propagation-seconds={propagation_seconds}"
297+
base_cmd.extend([propagation_param])
295298

296299
# Log command with masked email for debugging
297300
masked_cmd = [arg if not (i > 0 and base_cmd[i-1] == "--email") else "<email>"

0 commit comments

Comments
 (0)