Merge pull request #4825 from Dokploy/fix/allow-clearing-server-domain

fix(settings): allow clearing the server domain
This commit is contained in:
Mauricio Siu
2026-07-14 15:20:27 -06:00
committed by GitHub
@@ -43,7 +43,8 @@ const addServerDomain = z
.string() .string()
.trim() .trim()
.toLowerCase() .toLowerCase()
.refine((val) => VALID_HOSTNAME_REGEX.test(val), { // empty clears the server domain and reverts to IP-only access
.refine((val) => val === "" || VALID_HOSTNAME_REGEX.test(val), {
message: INVALID_HOSTNAME_MESSAGE, message: INVALID_HOSTNAME_MESSAGE,
}), }),
letsEncryptEmail: z.string(), letsEncryptEmail: z.string(),
@@ -51,7 +52,7 @@ const addServerDomain = z
certificateType: z.enum(["letsencrypt", "none", "custom"]), certificateType: z.enum(["letsencrypt", "none", "custom"]),
}) })
.superRefine((data, ctx) => { .superRefine((data, ctx) => {
if (data.https && !data.certificateType) { if (data.domain && data.https && !data.certificateType) {
ctx.addIssue({ ctx.addIssue({
code: z.ZodIssueCode.custom, code: z.ZodIssueCode.custom,
path: ["certificateType"], path: ["certificateType"],
@@ -59,6 +60,7 @@ const addServerDomain = z
}); });
} }
if ( if (
data.domain &&
data.https && data.https &&
data.certificateType === "letsencrypt" && data.certificateType === "letsencrypt" &&
!data.letsEncryptEmail !data.letsEncryptEmail