From ee74df3d0768c6b2f468d0daf3729d5b0dc34aeb Mon Sep 17 00:00:00 2001 From: Presley Pizzo <1290996+presleyp@users.noreply.github.com> Date: Mon, 5 Dec 2022 17:19:30 -0500 Subject: [PATCH] Fix scope of dbTTL (#5197) --- coderd/workspaces.go | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/coderd/workspaces.go b/coderd/workspaces.go index d46b0d7d2c..f69a98b8f0 100644 --- a/coderd/workspaces.go +++ b/coderd/workspaces.go @@ -672,10 +672,11 @@ func (api *API) putWorkspaceTTL(rw http.ResponseWriter, r *http.Request) { var dbTTL sql.NullInt64 err := api.Database.InTx(func(s database.Store) error { + var validityErr error // don't override 0 ttl with template default here because it indicates disabled auto-stop - dbTTL, err := validWorkspaceTTLMillis(req.TTLMillis, 0) - if err != nil { - return codersdk.ValidationError{Field: "ttl_ms", Detail: err.Error()} + dbTTL, validityErr = validWorkspaceTTLMillis(req.TTLMillis, 0) + if validityErr != nil { + return codersdk.ValidationError{Field: "ttl_ms", Detail: validityErr.Error()} } if err := s.UpdateWorkspaceTTL(ctx, database.UpdateWorkspaceTTLParams{ ID: workspace.ID,