Swarm Settings don't reset to default value correctly #4553

Activity

1. d4rky-pl

   added

   needs-triage🔍

   on Jun 5, 2026
2. dosubot

   added

   bugSomething isn't workingSomething isn't working

   on Jun 5, 2026
3. dosubot commented on Jun 5, 2026

   

   dosubotBot

   on Jun 5, 2026 – with DosuBot

   More actions

   Hi @d4rky-pl! I'm Dosu and I'm helping the Dokploy team.

   The root cause is in the Zod schema used by the health check form. The schema uses z.coerce.number() for the numeric fields (Interval, Timeout, StartPeriod, Retries), which converts empty strings (what HTML5 number inputs produce when cleared) to 0 instead of undefined [1].

   This breaks the form's existing hasAnyValue detection logic [2]:

   const hasAnyValue =
     (formData.Test && formData.Test.length > 0) ||
     formData.Interval !== undefined ||
     formData.Timeout !== undefined ||  // This is 0 (truthy check passes) instead of undefined
     formData.StartPeriod !== undefined ||
     formData.Retries !== undefined;

   Since the coerced 0 values pass the !== undefined checks, the form sends { Timeout: 0, Test: [], ... } to the database instead of null (which would clear the configuration entirely).

   Suggested fix — replace z.coerce.number() with a transform that treats empty strings as undefined:

   const optionalNumber = z
     .union([z.string(), z.number()])
     .transform((val) => (val === "" ? undefined : Number(val)))
     .optional();
   
   export const healthCheckFormSchema = z.object({
     Test: z.array(z.string()).optional(),
     Interval: optionalNumber,
     Timeout: optionalNumber,
     StartPeriod: optionalNumber,
     Retries: optionalNumber,
   });

   This preserves the existing hasAnyValue logic and correctly results in null being sent to the API when all fields are cleared. The file to modify is apps/dokploy/components/dashboard/application/advanced/cluster/swarm-forms/health-check-form.tsx [3].

   To reply, just mention @dosu.

   ---

   Docs are dead. Just use Dosu.

   
4. Siumauricio

   closed this as completedin #4558on Jun 6, 2026
5. Siumauricio

   added a commit that references this issue on Jun 6, 2026

   fix: swarm health check fields not resetting to default values (#4558)

   ...

   This issue will close once commit 57ef96a is merged into the 'canary' branch.

   57ef96a
6. Siumauricio

   removed

   needs-triage🔍

   on Jun 6, 2026
7. Siumauricio

   added a commit that references this issue on Jun 8, 2026

   🚀 Release v0.29.8 (#4562)

   ...

   This issue will close once commit b3c2e1e is merged into the 'canary' branch.

   b3c2e1e
8. github-actions

   added a commit that references this issue on Jul 13, 2026

   chore(release): 0.37.0 [skip ci]

   ...

   e8c1e68