coder
diff --git a/‎agent/apphealth.go
+5-4 b/‎agent/apphealth.go
+5-4
diff --git a/‎coderd/database/databasefake/databasefake.go
+1 b/‎coderd/database/databasefake/databasefake.go
+1
diff --git a/‎coderd/database/migrations/000087_external_app_url.down.sql
+1 b/‎coderd/database/migrations/000087_external_app_url.down.sql
+1
diff --git a/‎coderd/database/queries.sql.go
+4-1 b/‎coderd/database/queries.sql.go
+4-1
diff --git a/‎coderd/database/queries/workspaceapps.sql
+2-1 b/‎coderd/database/queries/workspaceapps.sql
+2-1
diff --git a/‎coderd/provisionerdserver/provisionerdserver.go
+1 b/‎coderd/provisionerdserver/provisionerdserver.go
+1
diff --git a/‎coderd/workspaceagents.go
+1 b/‎coderd/workspaceagents.go
+1
diff --git a/‎provisioner/terraform/resources.go
+2 b/‎provisioner/terraform/resources.go
+2
@@ -34,10 +34,11 @@ func NewWorkspaceAppHealthReporter(logger slog.Logger, apps []codersdk.Workspace
 		hasHealthchecksEnabled := false
 		health := make(map[uuid.UUID]codersdk.WorkspaceAppHealth, 0)
 		for _, app := range apps {
-			health[app.ID] = app.Health
-			if !hasHealthchecksEnabled && app.Health != codersdk.WorkspaceAppHealthDisabled {
-				hasHealthchecksEnabled = true
+			if app.Health == codersdk.WorkspaceAppHealthDisabled {
+				continue
 			}
+			health[app.ID] = app.Health
+			hasHealthchecksEnabled = true
 		}
 
 		// no need to run this loop if no health checks are configured.
@@ -77,7 +78,7 @@ func NewWorkspaceAppHealthReporter(logger slog.Logger, apps []codersdk.Workspace
 							return err
 						}
 						// successful healthcheck is a non-5XX status code
-						res.Body.Close()
+						_ = res.Body.Close()
 						if res.StatusCode >= http.StatusInternalServerError {
 							return xerrors.Errorf("error status code: %d", res.StatusCode)
 						}
 
@@ -2743,6 +2743,7 @@ func (q *fakeQuerier) InsertWorkspaceApp(_ context.Context, arg database.InsertW
 		Icon:                 arg.Icon,
 		Command:              arg.Command,
 		Url:                  arg.Url,
+		External:             arg.External,
 		Subdomain:            arg.Subdomain,
 		SharingLevel:         arg.SharingLevel,
 		HealthcheckUrl:       arg.HealthcheckUrl,
 
@@ -0,0 +1 @@
+ALTER TABLE workspace_apps DROP COLUMN external;
@@ -21,6 +21,7 @@ INSERT INTO
         icon,
         command,
         url,
+        external,
         subdomain,
         sharing_level,
         healthcheck_url,
@@ -29,7 +30,7 @@ INSERT INTO
         health
     )
 VALUES
-    ($1, $2, $3, $4, $5, $6, $7, $8, $9, $10, $11, $12, $13, $14) RETURNING *;
+    ($1, $2, $3, $4, $5, $6, $7, $8, $9, $10, $11, $12, $13, $14, $15) RETURNING *;
 
 -- name: UpdateWorkspaceAppHealthByID :exec
 UPDATE
 
@@ -952,6 +952,7 @@ func InsertWorkspaceResource(ctx context.Context, db database.Store, jobID uuid.
 					String: app.Url,
 					Valid:  app.Url != "",
 				},
+				External:             app.External,
 				Subdomain:            app.Subdomain,
 				SharingLevel:         sharingLevel,
 				HealthcheckUrl:       app.Healthcheck.Url,
 
@@ -645,6 +645,7 @@ func convertApps(dbApps []database.WorkspaceApp) []codersdk.WorkspaceApp {
 		apps = append(apps, codersdk.WorkspaceApp{
 			ID:           dbApp.ID,
 			URL:          dbApp.Url.String,
+			External:     dbApp.External,
 			Slug:         dbApp.Slug,
 			DisplayName:  dbApp.DisplayName,
 			Command:      dbApp.Command.String,
 
@@ -38,6 +38,7 @@ type agentAppAttributes struct {
 	Name        string                     `mapstructure:"name"`
 	Icon        string                     `mapstructure:"icon"`
 	URL         string                     `mapstructure:"url"`
+	External    bool                       `mapstructure:"external"`
 	Command     string                     `mapstructure:"command"`
 	Share       string                     `mapstructure:"share"`
 	Subdomain   bool                       `mapstructure:"subdomain"`
@@ -297,6 +298,7 @@ func ConvertResources(module *tfjson.StateModule, rawGraph string) ([]*proto.Res
 					Slug:         attrs.Slug,
 					DisplayName:  attrs.DisplayName,
 					Command:      attrs.Command,
+					External:     attrs.External,
 					Url:          attrs.URL,
 					Icon:         attrs.Icon,
 					Subdomain:    attrs.Subdomain,
Original file line number	Diff line number	Diff line change
`@@ -34,10 +34,11 @@ func NewWorkspaceAppHealthReporter(logger slog.Logger, apps []codersdk.Workspace`
`34`	`34`	`hasHealthchecksEnabled := false`
`35`	`35`	`health := make(map[uuid.UUID]codersdk.WorkspaceAppHealth, 0)`
`36`	`36`	`for _, app := range apps {`
`37`		`- health[app.ID] = app.Health`
`38`		`- if !hasHealthchecksEnabled && app.Health != codersdk.WorkspaceAppHealthDisabled {`
`39`		`- hasHealthchecksEnabled = true`
	`37`	`+ if app.Health == codersdk.WorkspaceAppHealthDisabled {`
	`38`	`+ continue`
`40`	`39`	`}`
	`40`	`+ health[app.ID] = app.Health`
	`41`	`+ hasHealthchecksEnabled = true`
`41`	`42`	`}`
`42`	`43`
`43`	`44`	`// no need to run this loop if no health checks are configured.`
`@@ -77,7 +78,7 @@ func NewWorkspaceAppHealthReporter(logger slog.Logger, apps []codersdk.Workspace`
`77`	`78`	`return err`
`78`	`79`	`}`
`79`	`80`	`// successful healthcheck is a non-5XX status code`
`80`		`- res.Body.Close()`
	`81`	`+ _ = res.Body.Close()`
`81`	`82`	`if res.StatusCode >= http.StatusInternalServerError {`
`82`	`83`	`return xerrors.Errorf("error status code: %d", res.StatusCode)`
`83`	`84`	`}`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	`+ALTER TABLE workspace_apps DROP COLUMN external;`