Update server_base_path format logic in modules

This change addresses the improvement of readability and consistency by simplifying the `server_base_path` construction logic, thereby enhancing maintainability across the respective Terraform modules.

filebrowser/README.md

@@ -14,7 +14,7 @@ A file browser for your workspace.
 ```tf
 module "filebrowser" {
   source   = "registry.coder.com/modules/filebrowser/coder"
-  version  = "1.0.22"
+  version  = "1.0.23"
   agent_id = coder_agent.example.id
 }
 ```
@@ -28,7 +28,7 @@ module "filebrowser" {
 ```tf
 module "filebrowser" {
   source   = "registry.coder.com/modules/filebrowser/coder"
-  version  = "1.0.22"
+  version  = "1.0.23"
   agent_id = coder_agent.example.id
   folder   = "/home/coder/project"
 }
@@ -39,17 +39,29 @@ module "filebrowser" {
 ```tf
 module "filebrowser" {
   source        = "registry.coder.com/modules/filebrowser/coder"
-  version       = "1.0.22"
+  version       = "1.0.23"
   agent_id      = coder_agent.example.id
   database_path = ".config/filebrowser.db"
 }
 ```
 
-### Serve from the same domain (no subdomain)
+### Serve on a subpath (no wildcard subdomain)
 
 ```tf
 module "filebrowser" {
   source    = "registry.coder.com/modules/filebrowser/coder"
+  version   = "1.0.23"
+  agent_id  = coder_agent.example.id
+  subdomain = false
+}
+```
+
+### Serve on a subpath with a specific agent name (multiple agents)
+
+```tf
+module "filebrowser" {
+  source     = "registry.coder.com/modules/filebrowser/coder"
+  version    = "1.0.23"
   agent_id   = coder_agent.example.id
   agent_name = "main"
   subdomain  = false

filebrowser/main.tf

@@ -20,13 +20,8 @@ data "coder_workspace_owner" "me" {}
 
 variable "agent_name" {
   type        = string
-  description = "The name of the main deployment. (Used to build the subpath for coder_app.)"
+  description = "The name of the coder_agent resource. (Only required if subdomain is false and the template uses multiple agents.)"
-  default     = ""
+  default     = null
-  validation {
-    # If subdomain is false, then agent_name must be set.
-    condition     = var.subdomain || var.agent_name != ""
-    error_message = "The agent_name must be set."
-  }
 }
 
 variable "database_path" {
@@ -73,6 +68,12 @@ variable "order" {
   default     = null
 }
 
+variable "slug" {
+  type        = string
+  description = "The slug of the coder_app resource."
+  default     = "filebrowser"
+}
+
 variable "subdomain" {
   type        = bool
   description = <<-EOT
@@ -85,7 +86,7 @@ variable "subdomain" {
 resource "coder_script" "filebrowser" {
   agent_id     = var.agent_id
   display_name = "File Browser"
-  icon         = "https://raw.githubusercontent.com/filebrowser/logo/master/icon_raw.svg"
+  icon         = "/icon/filebrowser.svg"
   script = templatefile("${path.module}/run.sh", {
     LOG_PATH : var.log_path,
     PORT : var.port,
@@ -93,18 +94,30 @@ resource "coder_script" "filebrowser" {
     LOG_PATH : var.log_path,
     DB_PATH : var.database_path,
     SUBDOMAIN : var.subdomain,
-    SERVER_BASE_PATH : var.subdomain ? "" : format("/@%s/%s.%s/apps/filebrowser", data.coder_workspace_owner.me.name, data.coder_workspace.me.name, var.agent_name),
+    SERVER_BASE_PATH : local.server_base_path
   })
   run_on_start = true
 }
 
 resource "coder_app" "filebrowser" {
   agent_id     = var.agent_id
-  slug         = "filebrowser"
+  slug         = var.slug
   display_name = "File Browser"
-  url          = "http://localhost:${var.port}"
+  url          = local.url
-  icon         = "https://raw.githubusercontent.com/filebrowser/logo/master/icon_raw.svg"
+  icon         = "/icon/filebrowser.svg"
   subdomain    = var.subdomain
   share        = var.share
   order        = var.order
+
+  healthcheck {
+    url       = local.healthcheck_url
+    interval  = 5
+    threshold = 6
+  }
+}
+
+locals {
+  server_base_path = var.subdomain ? "" : format("/@%s/%s%s/apps/%s", data.coder_workspace_owner.me.name, data.coder_workspace.me.name, var.agent_name != null ? ".${var.agent_name}" : "", var.slug)
+  url              = "http://localhost:${var.port}${local.server_base_path}"
+  healthcheck_url  = "http://localhost:${var.port}${local.server_base_path}/health"
 }

jetbrains-gateway/README.md

@@ -14,7 +14,7 @@ This module adds a JetBrains Gateway Button to open any workspace with a single
 ```tf
 module "jetbrains_gateway" {
   source         = "registry.coder.com/modules/jetbrains-gateway/coder"
-  version        = "1.0.21"
+  version        = "1.0.23"
   agent_id       = coder_agent.example.id
   agent_name     = "example"
   folder         = "/home/coder/example"
@@ -32,7 +32,7 @@ module "jetbrains_gateway" {
 ```tf
 module "jetbrains_gateway" {
   source         = "registry.coder.com/modules/jetbrains-gateway/coder"
-  version        = "1.0.21"
+  version        = "1.0.23"
   agent_id       = coder_agent.example.id
   agent_name     = "example"
   folder         = "/home/coder/example"
@@ -46,7 +46,7 @@ module "jetbrains_gateway" {
 ```tf
 module "jetbrains_gateway" {
   source         = "registry.coder.com/modules/jetbrains-gateway/coder"
-  version        = "1.0.21"
+  version        = "1.0.23"
   agent_id       = coder_agent.example.id
   agent_name     = "example"
   folder         = "/home/coder/example"
@@ -61,7 +61,7 @@ module "jetbrains_gateway" {
 ```tf
 module "jetbrains_gateway" {
   source         = "registry.coder.com/modules/jetbrains-gateway/coder"
-  version        = "1.0.21"
+  version        = "1.0.23"
   agent_id       = coder_agent.example.id
   agent_name     = "example"
   folder         = "/home/coder/example"

jetbrains-gateway/main.tf

@@ -20,13 +20,13 @@ variable "agent_id" {
 
 variable "slug" {
   type        = string
-  description = "The slug for the coder_app. Allows resuing the module with the same template."
+  description = "The slug for the coder_app resource."
   default     = "gateway"
 }
 
 variable "agent_name" {
   type        = string
-  description = "Agent name."
+  description = "The name of the coder_agent resource."
 }
 
 variable "folder" {
@@ -258,26 +258,18 @@ resource "coder_app" "gateway" {
   icon         = local.icon
   external     = true
   order        = var.order
-  url = join("", [
+  url = format(
-    "jetbrains-gateway://connect#type=coder&workspace=",
+    "jetbrains-gateway://connect#type=coder&workspace=%s&owner=%s&agent=%s&folder=%s&url=%s&token=%s&ide_product_code=%s&ide_build_number=%s&ide_download_link=%s",
     data.coder_workspace.me.name,
-    "&owner=",
     data.coder_workspace_owner.me.name,
-    "&agent=",
     var.agent_name,
-    "&folder=",
     var.folder,
-    "&url=",
     data.coder_workspace.me.access_url,
-    "&token=",
     "$SESSION_TOKEN",
-    "&ide_product_code=",
     data.coder_parameter.jetbrains_ide.value,
-    "&ide_build_number=",
     local.build_number,
-    "&ide_download_link=",
+    local.download_link
-    local.download_link,
+  )
-  ])
 }
 
 output "identifier" {

jupyter-notebook/README.md

@@ -11,12 +11,36 @@ tags: [jupyter, helper, ide, web]
 
 A module that adds Jupyter Notebook in your Coder template.
 
+```tf
+module "jupyter-notebook" {
+  source   = "registry.coder.com/modules/jupyter-notebook/coder"
+  version  = "1.0.23"
+  agent_id = coder_agent.example.id
+}
+```
+
 ![Jupyter Notebook](../.images/jupyter-notebook.png)
 
+## Examples
+
+### Serve on a subpath (no wildcard subdomain)
+
 ```tf
 module "jupyter-notebook" {
   source    = "registry.coder.com/modules/jupyter-notebook/coder"
-  version  = "1.0.19"
+  version   = "1.0.23"
   agent_id  = coder_agent.example.id
+  subdomain = false
+}
+```
+
+### Serve on a subpath with a specific agent name (multiple agents)
+
+```tf
+module "jupyter-notebook" {
+  source     = "registry.coder.com/modules/jupyter-notebook/coder"
+  version    = "1.0.23"
+  agent_id   = coder_agent.example.id
+  agent_name = "main"
 }
 ```

jupyter-notebook/main.tf

@@ -42,9 +42,30 @@ variable "order" {
   default     = null
 }
 
+variable "agent_name" {
+  type        = string
+  description = "The name of the coder_agent resource. (Only required if subdomain is false and the template uses multiple agents.)"
+  default     = null
+}
+
+variable "slug" {
+  type        = string
+  description = "The slug of the coder_app resource."
+  default     = "jupyter-notebook"
+}
+
+variable "subdomain" {
+  type        = bool
+  description = <<-EOT
+    Determines whether the app will be accessed via it's own subdomain or whether it will be accessed via a path on Coder.
+    If wildcards have not been setup by the administrator then apps with "subdomain" set to true will not be accessible.
+  EOT
+  default     = true
+}
+
 resource "coder_script" "jupyter-notebook" {
   agent_id     = var.agent_id
-  display_name = "jupyter-notebook"
+  display_name = "Jupyter Notebook"
   icon         = "/icon/jupyter.svg"
   script = templatefile("${path.module}/run.sh", {
     LOG_PATH : var.log_path,
@@ -55,11 +76,26 @@ resource "coder_script" "jupyter-notebook" {
 
 resource "coder_app" "jupyter-notebook" {
   agent_id     = var.agent_id
-  slug         = "jupyter-notebook"
+  slug         = var.slug
   display_name = "Jupyter Notebook"
-  url          = "http://localhost:${var.port}"
+  url          = local.url
   icon         = "/icon/jupyter.svg"
   subdomain    = true
   share        = var.share
   order        = var.order
+
+  healthcheck {
+    url       = local.healthcheck_url
+    interval  = 5
+    threshold = 6
+  }
+}
+
+data "coder_workspace_owner" "me" {}
+data "coder_workspace" "me" {}
+
+locals {
+  server_base_path = var.subdomain ? "" : format("/@%s/%s%s/apps/%s", data.coder_workspace_owner.me.name, data.coder_workspace.me.name, var.agent_name != null ? ".${var.agent_name}" : "", var.slug)
+  url              = "http://localhost:${var.port}${local.server_base_path}"
+  healthcheck_url  = "http://localhost:${var.port}${local.server_base_path}/api"
 }

jupyterlab/README.md

@@ -11,12 +11,36 @@ tags: [jupyter, helper, ide, web]
 
 A module that adds JupyterLab in your Coder template.
 
+```tf
+module "jupyterlab" {
+  source   = "registry.coder.com/modules/jupyterlab/coder"
+  version  = "1.0.23"
+  agent_id = coder_agent.example.id
+}
+```
+
 ![JupyterLab](../.images/jupyterlab.png)
 
+## Examples
+
+### Serve on a subpath (no wildcard subdomain)
+
 ```tf
 module "jupyterlab" {
   source    = "registry.coder.com/modules/jupyterlab/coder"
-  version  = "1.0.22"
+  version   = "1.0.23"
   agent_id  = coder_agent.example.id
+  subdomain = false
+}
+```
+
+### Serve on a subpath with a specific agent name (multiple agents)
+
+```tf
+module "jupyterlab" {
+  source     = "registry.coder.com/modules/jupyterlab/coder"
+  version    = "1.0.23"
+  agent_id   = coder_agent.example.id
+  agent_name = "main"
 }
 ```

jupyterlab/main.tf

@@ -41,7 +41,10 @@ variable "share" {
 
 variable "subdomain" {
   type        = bool
-  description = "Determines whether JupyterLab will be accessed via its own subdomain or whether it will be accessed via a path on Coder."
+  description = <<-EOT
+    Determines whether the app will be accessed via it's own subdomain or whether it will be accessed via a path on Coder.
+    If wildcards have not been setup by the administrator then apps with "subdomain" set to true will not be accessible.
+  EOT
   default     = true
 }
 
@@ -51,6 +54,18 @@ variable "order" {
   default     = null
 }
 
+variable "agent_name" {
+  type        = string
+  description = "The name of the coder_agent resource. (Only required if subdomain is false and the template uses multiple agents.)"
+  default     = null
+}
+
+variable "slug" {
+  type        = string
+  description = "The slug of the coder_app resource."
+  default     = "jupyterlab"
+}
+
 resource "coder_script" "jupyterlab" {
   agent_id     = var.agent_id
   display_name = "jupyterlab"
@@ -58,18 +73,24 @@ resource "coder_script" "jupyterlab" {
   script = templatefile("${path.module}/run.sh", {
     LOG_PATH : var.log_path,
     PORT : var.port
-    BASE_URL : var.subdomain ? "" : "/@${data.coder_workspace_owner.me.name}/${data.coder_workspace.me.name}/apps/jupyterlab"
+    BASE_URL : local.server_base_path
   })
   run_on_start = true
 }
 
 resource "coder_app" "jupyterlab" {
   agent_id     = var.agent_id
-  slug         = "jupyterlab" # sync with the usage in URL
+  slug         = var.slug
   display_name = "JupyterLab"
-  url          = var.subdomain ? "http://localhost:${var.port}" : "http://localhost:${var.port}/@${data.coder_workspace_owner.me.name}/${data.coder_workspace.me.name}/apps/jupyterlab"
+  url          = local.url
   icon         = "/icon/jupyter.svg"
   subdomain    = var.subdomain
   share        = var.share
   order        = var.order
 }
+
+locals {
+  server_base_path = var.subdomain ? "" : format("/@%s/%s%s/apps/%s", data.coder_workspace_owner.me.name, data.coder_workspace.me.name, var.agent_name != null ? ".${var.agent_name}" : "", var.slug)
+  url              = "http://localhost:${var.port}${local.server_base_path}"
+  healthcheck_url  = "http://localhost:${var.port}${local.server_base_path}/api"
+}

kasmvnc/README.md

@@ -14,29 +14,10 @@ Automatically install [KasmVNC](https://kasmweb.com/kasmvnc) in a workspace, and
 ```tf
 module "kasmvnc" {
   source              = "registry.coder.com/modules/kasmvnc/coder"
-  version             = "1.0.23"
+  version             = "1.0.22"
   agent_id            = coder_agent.example.id
   desktop_environment = "xfce"
 }
 ```
 
 > **Note:** This module only works on workspaces with a pre-installed desktop environment. As an example base image you can use `codercom/enterprise-desktop` image.
-
-> **Note:** You can also use the kasmtech [custom images](https://kasmweb.com/docs/latest/guide/custom_images.html) by extending them as following:
-
-```Dockerfile
-FROM kasmweb/postman:1.16.0
-ARG USER=kasm-user
-USER root
-# Overwrite the existing config file to disable ssl
-RUN cat <<EOF > /etc/kasmvnc/kasmvnc.yaml
-network:
-  protocol: http
-  ssl:
-    require_ssl: false
-  udp:
-    public_ip: 127.0.0.1
-EOF
-RUN addgroup $USER ssl-cert
-USER $USER
-```

kasmvnc/run.sh

@@ -5,7 +5,7 @@
 # Function to check if vncserver is already installed
 check_installed() {
   if command -v vncserver &> /dev/null; then
-    echo "A binary with name vncserver already installed."
+    echo "vncserver is already installed."
     return 0 # Don't exit, just indicate it's installed
   else
     return 1 # Indicates not installed
@@ -34,7 +34,7 @@ install_deb() {
   download_file $url /tmp/kasmvncserver.deb
   sudo apt-get update
   DEBIAN_FRONTEND=noninteractive sudo apt-get install --yes -qq --no-install-recommends --no-install-suggests /tmp/kasmvncserver.deb
-  sudo usermod -aG ssl-cert $USER
+  sudo adduser $USER ssl-cert
   rm /tmp/kasmvncserver.deb
 }
 
@@ -75,35 +75,35 @@ install_alpine() {
   rm /tmp/kasmvncserver.tgz
 }
 
-# Check if vncserver is installed, and install if not
+# Detect system information
-if ! check_installed; then
+distro=$(grep "^ID=" /etc/os-release | awk -F= '{print $2}')
-  # Detect system information
+version=$(grep "^VERSION_ID=" /etc/os-release | awk -F= '{print $2}' | tr -d '"')
-  distro=$(grep "^ID=" /etc/os-release | awk -F= '{print $2}')
+arch=$(uname -m)
-  version=$(grep "^VERSION_ID=" /etc/os-release | awk -F= '{print $2}' | tr -d '"')
-  arch=$(uname -m)
 
-  echo "Detected Distribution: $distro"
+echo "Detected Distribution: $distro"
-  echo "Detected Version: $version"
+echo "Detected Version: $version"
-  echo "Detected Architecture: $arch"
+echo "Detected Architecture: $arch"
 
-  # Map arch to package arch
+# Map arch to package arch
-  if [[ "$arch" == "x86_64" ]]; then
+if [[ "$arch" == "x86_64" ]]; then
   if [[ "$distro" == "ubuntu" || "$distro" == "debian" || "$distro" == "kali" ]]; then
     arch="amd64"
   else
     arch="x86_64"
   fi
-  elif [[ "$arch" == "aarch64" || "$arch" == "arm64" ]]; then
+elif [[ "$arch" == "aarch64" || "$arch" == "arm64" ]]; then
   if [[ "$distro" == "ubuntu" || "$distro" == "debian" || "$distro" == "kali" ]]; then
     arch="arm64"
   else
     arch="aarch64"
   fi
-  else
+else
   echo "Unsupported architecture: $arch"
   exit 1
-  fi
+fi
 
+# Check if vncserver is installed, and install if not
+if ! check_installed; then
   echo "Installing KASM version: ${VERSION}"
   case $distro in
     ubuntu | debian | kali)
@@ -155,12 +155,10 @@ if ! check_installed; then
       ;;
   esac
 else
-  echo "Skipping installation."
+  echo "vncserver already installed. Skipping installation."
 fi
 
-# Try to create /etc/kasmvnc/kasmvnc.yaml system-wide
+# Coder port-forwarding from dashboard only supports HTTP
-# we don't fail as some images might be missing sudo permissions
-sudo mkdir -p /etc/kasmvnc || true
 sudo bash -c "cat > /etc/kasmvnc/kasmvnc.yaml <<EOF
 network:
   protocol: http
@@ -169,22 +167,7 @@ network:
     require_ssl: false
   udp:
     public_ip: 127.0.0.1
-EOF" || true
+EOF"
-
-
-# There could already be a config file in the image at /etc/kasmvnc/kasmvnc.yaml, but we need to set the websocket port
-mkdir -p "$HOME/.vnc"
-cat > "$HOME/.vnc/kasmvnc.yaml" <<EOF
-network:
-  protocol: http
-  websocket_port: ${PORT}
-  ssl:
-    require_ssl: false
-    pem_certificate:
-    pem_key:
-  udp:
-    public_ip: 127.0.0.1
-EOF
 
 # This password is not used since we start the server without auth.
 # The server is protected via the Coder session token / tunnel
@@ -193,4 +176,4 @@ echo -e "password\npassword\n" | vncpasswd -wo -u $USER
 
 # Start the server
 printf "🚀 Starting KasmVNC server...\n"
-vncserver -select-de ${DESKTOP_ENVIRONMENT} -disableBasicAuth > /tmp/kasmvncserver.log 2>&1 &
+sudo -u $USER bash -c "vncserver -select-de ${DESKTOP_ENVIRONMENT} -disableBasicAuth" > /tmp/kasmvncserver.log 2>&1 &

vscode-desktop/README.md

@@ -16,7 +16,7 @@ Uses the [Coder Remote VS Code Extension](https://github.com/coder/vscode-coder)
 ```tf
 module "vscode" {
   source   = "registry.coder.com/modules/vscode-desktop/coder"
-  version  = "1.0.15"
+  version  = "1.0.23"
   agent_id = coder_agent.example.id
 }
 ```
@@ -28,7 +28,7 @@ module "vscode" {
 ```tf
 module "vscode" {
   source   = "registry.coder.com/modules/vscode-desktop/coder"
-  version  = "1.0.15"
+  version  = "1.0.23"
   agent_id = coder_agent.example.id
   folder   = "/home/coder/project"
 }

vscode-desktop/main.tf

@@ -20,6 +20,12 @@ variable "folder" {
   default     = ""
 }
 
+variable "slug" {
+  type        = string
+  description = "The slug of the coder_app resource."
+  default     = "vscode-desktop"
+}
+
 variable "open_recent" {
   type        = bool
   description = "Open the most recent workspace or folder. Falls back to the folder if there is no recent workspace or folder to open."
@@ -39,21 +45,17 @@ resource "coder_app" "vscode" {
   agent_id     = var.agent_id
   external     = true
   icon         = "/icon/code.svg"
-  slug         = "vscode"
+  slug         = var.slug
   display_name = "VS Code Desktop"
   order        = var.order
-  url = join("", [
+  url = format(
-    "vscode://coder.coder-remote/open",
+    "vscode://coder.coder-remote/open?owner=%s&workspace=%s%s%s&url=%s&token=$SESSION_TOKEN",
-    "?owner=",
     data.coder_workspace_owner.me.name,
-    "&workspace=",
     data.coder_workspace.me.name,
-    var.folder != "" ? join("", ["&folder=", var.folder]) : "",
+    var.folder != "" ? "&folder=${var.folder}" : "",
     var.open_recent ? "&openRecent" : "",
-    "&url=",
+    data.coder_workspace.me.access_url
-    data.coder_workspace.me.access_url,
+  )
-    "&token=$SESSION_TOKEN",
-  ])
 }
 
 output "vscode_url" {

vscode-web/README.md

@@ -14,7 +14,7 @@ Automatically install [Visual Studio Code Server](https://code.visualstudio.com/
 ```tf
 module "vscode-web" {
   source         = "registry.coder.com/modules/vscode-web/coder"
-  version        = "1.0.22"
+  version        = "1.0.23"
   agent_id       = coder_agent.example.id
   accept_license = true
 }
@@ -29,7 +29,7 @@ module "vscode-web" {
 ```tf
 module "vscode-web" {
   source         = "registry.coder.com/modules/vscode-web/coder"
-  version        = "1.0.22"
+  version        = "1.0.23"
   agent_id       = coder_agent.example.id
   install_prefix = "/home/coder/.vscode-web"
   folder         = "/home/coder"
@@ -42,7 +42,7 @@ module "vscode-web" {
 ```tf
 module "vscode-web" {
   source         = "registry.coder.com/modules/vscode-web/coder"
-  version        = "1.0.22"
+  version        = "1.0.23"
   agent_id       = coder_agent.example.id
   extensions     = ["github.copilot", "ms-python.python", "ms-toolsai.jupyter"]
   accept_license = true
@@ -56,7 +56,7 @@ Configure VS Code's [settings.json](https://code.visualstudio.com/docs/getstarte
 ```tf
 module "vscode-web" {
   source     = "registry.coder.com/modules/vscode-web/coder"
-  version    = "1.0.22"
+  version    = "1.0.23"
   agent_id   = coder_agent.example.id
   extensions = ["dracula-theme.theme-dracula"]
   settings = {
@@ -65,3 +65,26 @@ module "vscode-web" {
   accept_license = true
 }
 ```
+
+### Serve on a subpath (no wildcard subdomain)
+
+```tf
+module "vscode-web" {
+  source    = "registry.coder.com/modules/vscode-web/coder"
+  version   = "1.0.23"
+  agent_id  = coder_agent.example.id
+  subdomain = false
+}
+```
+
+### Serve on a subpath with a specific agent name (multiple agents)
+
+```tf
+module "vscode-web" {
+  source     = "registry.coder.com/modules/vscode-web/coder"
+  version    = "1.0.23"
+  agent_id   = coder_agent.example.id
+  agent_name = "main"
+  subdomain  = false
+}
+```

vscode-web/main.tf

@@ -130,6 +130,12 @@ variable "subdomain" {
   default     = true
 }
 
+variable "agent_name" {
+  type        = string
+  description = "The name of the coder_agent resource. (Only required if subdomain is false and the template uses multiple agents.)"
+  default     = null
+}
+
 data "coder_workspace_owner" "me" {}
 data "coder_workspace" "me" {}
 
@@ -185,7 +191,7 @@ resource "coder_app" "vscode-web" {
 }
 
 locals {
-  server_base_path = var.subdomain ? "" : format("/@%s/%s/apps/%s/", data.coder_workspace_owner.me.name, data.coder_workspace.me.name, var.slug)
+  server_base_path = var.subdomain ? "" : format("/@%s/%s%s/apps/%s", data.coder_workspace_owner.me.name, data.coder_workspace.me.name, var.agent_name != null ? ".${var.agent_name}" : "", var.slug)
   url              = var.folder == "" ? "http://localhost:${var.port}${local.server_base_path}" : "http://localhost:${var.port}${local.server_base_path}?folder=${var.folder}"
-  healthcheck_url  = var.subdomain ? "http://localhost:${var.port}/healthz" : "http://localhost:${var.port}${local.server_base_path}/healthz"
+  healthcheck_url  = "http://localhost:${var.port}${local.server_base_path}/healthz"
 }

windows-rdp/README.md

@@ -15,7 +15,7 @@ Enable Remote Desktop + a web based client on Windows workspaces, powered by [de
 # AWS example. See below for examples of using this module with other providers
 module "windows_rdp" {
   source      = "registry.coder.com/modules/windows-rdp/coder"
-  version     = "1.0.18"
+  version     = "1.0.23"
   count       = data.coder_workspace.me.start_count
   agent_id    = resource.coder_agent.main.id
   resource_id = resource.aws_instance.dev.id
@@ -33,7 +33,7 @@ module "windows_rdp" {
 ```tf
 module "windows_rdp" {
   source      = "registry.coder.com/modules/windows-rdp/coder"
-  version     = "1.0.18"
+  version     = "1.0.23"
   count       = data.coder_workspace.me.start_count
   agent_id    = resource.coder_agent.main.id
   resource_id = resource.aws_instance.dev.id
@@ -45,7 +45,7 @@ module "windows_rdp" {
 ```tf
 module "windows_rdp" {
   source      = "registry.coder.com/modules/windows-rdp/coder"
-  version     = "1.0.18"
+  version     = "1.0.23"
   count       = data.coder_workspace.me.start_count
   agent_id    = resource.coder_agent.main.id
   resource_id = resource.google_compute_instance.dev[0].id

windows-rdp/main.tf

@@ -39,6 +39,18 @@ variable "admin_password" {
   sensitive = true
 }
 
+variable "port" {
+  type        = number
+  description = "The port to run the Devolutions Gateway on."
+  default     = 7171
+}
+
+variable "slug" {
+  type        = string
+  description = "The slug of the coder_app resource."
+  default     = "web-rdp"
+}
+
 resource "coder_script" "windows-rdp" {
   agent_id     = var.agent_id
   display_name = "windows-rdp"
@@ -47,6 +59,7 @@ resource "coder_script" "windows-rdp" {
   script = templatefile("${path.module}/powershell-installation-script.tftpl", {
     admin_username = var.admin_username
     admin_password = var.admin_password
+    port           = var.port
 
     # Wanted to have this be in the powershell template file, but Terraform
     # doesn't allow recursive calls to the templatefile function. Have to feed
@@ -63,14 +76,14 @@ resource "coder_script" "windows-rdp" {
 resource "coder_app" "windows-rdp" {
   agent_id     = var.agent_id
   share        = var.share
-  slug         = "web-rdp"
+  slug         = var.slug
   display_name = "Web RDP"
-  url          = "http://localhost:7171"
+  url          = "http://localhost:${var.port}"
   icon         = "/icon/desktop.svg"
   subdomain    = true
 
   healthcheck {
-    url       = "http://localhost:7171"
+    url       = "http://localhost:${var.port}"
     interval  = 5
     threshold = 15
   }
@@ -80,7 +93,7 @@ resource "coder_app" "rdp-docs" {
   agent_id     = var.agent_id
   display_name = "Local RDP"
   slug         = "rdp-docs"
-  icon         = "https://raw.githubusercontent.com/matifali/logos/main/windows.svg"
+  icon         = "/icon/windows.svg"
-  url          = "https://coder.com/docs/ides/remote-desktops#rdp-desktop"
+  url          = "https://coder.com/docs/user-guides/workspace-access/remote-desktops#rdp-desktop"
   external     = true
 }

windows-rdp/powershell-installation-script.tftpl

@@ -47,7 +47,7 @@ Install-DGatewayPackage
 
 # Configure Devolutions Gateway
 $Hostname = "localhost"
-$HttpListener = New-DGatewayListener 'http://*:7171' 'http://*:7171'
+$HttpListener = New-DGatewayListener "http://*:${port}" "http://*:${port}"
 $WebApp = New-DGatewayWebAppConfig -Enabled $true -Authentication None
 $ConfigParams = @{
     Hostname = $Hostname