Skip to content

Commit e54ca31

Browse files
authored
Merge branch 'main' into jetbrains
2 parents 283bdc3 + 6b9d0d4 commit e54ca31

File tree

16 files changed

+77
-72
lines changed

16 files changed

+77
-72
lines changed

.icons/rdp.svg

Lines changed: 35 additions & 0 deletions
Loading

MAINTAINER.md

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -14,9 +14,7 @@ brew install go
1414
sudo apt install golang-go
1515
```
1616

17-
## Daily Tasks
18-
19-
### Review PRs
17+
## Reviewing a PR
2018

2119
Check that PRs have:
2220

@@ -26,7 +24,7 @@ Check that PRs have:
2624
- [ ] Formatted code (`bun run fmt`)
2725
- [ ] Avatar image for new namespaces (`avatar.png` or `avatar.svg` in `.images/`)
2826

29-
#### Version Guidelines
27+
### Version Guidelines
3028

3129
When reviewing PRs, ensure the version change follows semantic versioning:
3230

@@ -42,7 +40,7 @@ PRs should clearly indicate the version change (e.g., `v1.2.3 → v1.2.4`).
4240
go build ./cmd/readmevalidation && ./readmevalidation
4341
```
4442

45-
## Releases
43+
## Making a Release
4644

4745
### Create Release Tags
4846

@@ -99,5 +97,3 @@ status: "community" # or "partner", "official"
9997
- **Tests fail**: Ensure Docker with `--network=host`, check Terraform syntax
10098
- **Wrong file structure**: Use `./scripts/new_module.sh` for new modules
10199
- **Missing namespace avatar**: Must be `avatar.png` or `avatar.svg` in `.images/` directory
102-
103-
That's it. Keep it simple.

bun.lockb

-721 Bytes
Binary file not shown.

package.json

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -8,21 +8,21 @@
88
"update-version": "./update-version.sh"
99
},
1010
"devDependencies": {
11-
"@types/bun": "^1.2.9",
12-
"bun-types": "^1.1.23",
11+
"@types/bun": "^1.2.18",
12+
"bun-types": "^1.2.18",
1313
"gray-matter": "^4.0.3",
14-
"marked": "^12.0.2",
15-
"prettier": "^3.3.3",
16-
"prettier-plugin-sh": "^0.13.1",
14+
"marked": "^16.0.0",
15+
"prettier": "^3.6.2",
16+
"prettier-plugin-sh": "^0.18.0",
1717
"prettier-plugin-terraform-formatter": "^1.2.1"
1818
},
1919
"peerDependencies": {
20-
"typescript": "^5.5.4"
20+
"typescript": "^5.8.3"
2121
},
2222
"prettier": {
2323
"plugins": [
2424
"prettier-plugin-sh",
2525
"prettier-plugin-terraform-formatter"
2626
]
2727
}
28-
}
28+
}

registry/coder/modules/local-windows-rdp/README.md

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,24 +1,25 @@
11
---
2-
display_name: Windows RDP Desktop
2+
display_name: RDP Desktop
33
description: Enable RDP on Windows and add a one-click Coder Desktop button for seamless access
4-
icon: ../../../../.icons/desktop.svg
4+
icon: ../../../../.icons/rdp.svg
55
maintainer_github: coder
66
verified: true
77
supported_os: [windows]
8-
tags: [rdp, windows, desktop, remote]
8+
tags: [rdp, windows, desktop, local]
99
---
1010

1111
# Windows RDP Desktop
1212

1313
This module enables Remote Desktop Protocol (RDP) on Windows workspaces and adds a one-click button to launch RDP sessions directly through [Coder Desktop](https://coder.com/docs/user-guides/desktop). It provides a complete, standalone solution for RDP access, eliminating the need for manual configuration or port forwarding through the Coder CLI.
1414

15-
> **Note**: [Coder Desktop](https://coder.com/docs/user-guides/desktop) is required on client devices to use the Local Windows RDP access feature.
15+
> [!NOTE]
16+
> [Coder Desktop](https://coder.com/docs/user-guides/desktop) is required on client devices to use the Local Windows RDP access feature.
1617
1718
```tf
1819
module "rdp_desktop" {
1920
count = data.coder_workspace.me.start_count
2021
source = "registry.coder.com/coder/local-windows-rdp/coder"
21-
version = "1.0.0"
22+
version = "1.0.1"
2223
agent_id = coder_agent.main.id
2324
agent_name = coder_agent.main.name
2425
}
@@ -51,7 +52,7 @@ Uses default credentials (Username: `Administrator`, Password: `coderRDP!`):
5152
module "rdp_desktop" {
5253
count = data.coder_workspace.me.start_count
5354
source = "registry.coder.com/coder/local-windows-rdp/coder"
54-
version = "1.0.0"
55+
version = "1.0.1"
5556
agent_id = coder_agent.main.id
5657
agent_name = coder_agent.main.name
5758
}
@@ -65,7 +66,7 @@ Specify a custom display name for the `coder_app` button:
6566
module "rdp_desktop" {
6667
count = data.coder_workspace.me.start_count
6768
source = "registry.coder.com/coder/local-windows-rdp/coder"
68-
version = "1.0.0"
69+
version = "1.0.1"
6970
agent_id = coder_agent.windows.id
7071
agent_name = "windows"
7172
display_name = "Windows Desktop"

registry/coder/modules/local-windows-rdp/main.test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ describe("local-windows-rdp", async () => {
7373
expect(app).not.toBeNull();
7474
expect(app?.slug).toBe("rdp-desktop");
7575
expect(app?.display_name).toBe("RDP Desktop");
76-
expect(app?.icon).toBe("/icon/desktop.svg");
76+
expect(app?.icon).toBe("/icon/rdp.svg");
7777
expect(app?.external).toBe(true);
7878

7979
// Verify the URI format
@@ -95,7 +95,7 @@ describe("local-windows-rdp", async () => {
9595
// Verify the script was created
9696
expect(script).not.toBeNull();
9797
expect(script?.display_name).toBe("Configure RDP");
98-
expect(script?.icon).toBe("/icon/desktop.svg");
98+
expect(script?.icon).toBe("/icon/rdp.svg");
9999
expect(script?.run_on_start).toBe(true);
100100
expect(script?.run_on_stop).toBe(false);
101101

registry/coder/modules/local-windows-rdp/main.tf

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ data "coder_workspace" "me" {}
6060
resource "coder_script" "rdp_setup" {
6161
agent_id = var.agent_id
6262
display_name = "Configure RDP"
63-
icon = "/icon/desktop.svg"
63+
icon = "/icon/rdp.svg"
6464
script = templatefile("${path.module}/configure-rdp.ps1", {
6565
username = var.username
6666
password = var.password
@@ -73,9 +73,8 @@ resource "coder_app" "rdp_desktop" {
7373
slug = "rdp-desktop"
7474
display_name = var.display_name
7575
url = "coder://${local.server_name}/v0/open/ws/${data.coder_workspace.me.name}/agent/${var.agent_name}/rdp?username=${var.username}&password=${var.password}"
76-
icon = "/icon/desktop.svg"
76+
icon = "/icon/rdp.svg"
7777
external = true
7878
order = var.order
7979
group = var.group
8080
}
81-
Lines changed: 15 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
---
2-
display_name: Windows RDP
2+
display_name: RDP Web
33
description: RDP Server and Web Client, powered by Devolutions Gateway
44
icon: ../../../../.icons/desktop.svg
55
maintainer_github: coder
@@ -14,11 +14,10 @@ Enable Remote Desktop + a web based client on Windows workspaces, powered by [de
1414
```tf
1515
# AWS example. See below for examples of using this module with other providers
1616
module "windows_rdp" {
17-
count = data.coder_workspace.me.start_count
18-
source = "registry.coder.com/coder/windows-rdp/coder"
19-
version = "1.2.1"
20-
agent_id = resource.coder_agent.main.id
21-
resource_id = resource.aws_instance.dev.id
17+
count = data.coder_workspace.me.start_count
18+
source = "registry.coder.com/coder/windows-rdp/coder"
19+
version = "1.2.2"
20+
agent_id = resource.coder_agent.main.id
2221
}
2322
```
2423

@@ -32,23 +31,21 @@ module "windows_rdp" {
3231

3332
```tf
3433
module "windows_rdp" {
35-
count = data.coder_workspace.me.start_count
36-
source = "registry.coder.com/coder/windows-rdp/coder"
37-
version = "1.2.1"
38-
agent_id = resource.coder_agent.main.id
39-
resource_id = resource.aws_instance.dev.id
34+
count = data.coder_workspace.me.start_count
35+
source = "registry.coder.com/coder/windows-rdp/coder"
36+
version = "1.2.2"
37+
agent_id = resource.coder_agent.main.id
4038
}
4139
```
4240

4341
### With Google Cloud
4442

4543
```tf
4644
module "windows_rdp" {
47-
count = data.coder_workspace.me.start_count
48-
source = "registry.coder.com/coder/windows-rdp/coder"
49-
version = "1.2.1"
50-
agent_id = resource.coder_agent.main.id
51-
resource_id = resource.google_compute_instance.dev[0].id
45+
count = data.coder_workspace.me.start_count
46+
source = "registry.coder.com/coder/windows-rdp/coder"
47+
version = "1.2.2"
48+
agent_id = resource.coder_agent.main.id
5249
}
5350
```
5451

@@ -58,13 +55,8 @@ module "windows_rdp" {
5855
module "windows_rdp" {
5956
count = data.coder_workspace.me.start_count
6057
source = "registry.coder.com/coder/windows-rdp/coder"
61-
version = "1.2.1"
58+
version = "1.2.2"
6259
agent_id = resource.coder_agent.main.id
63-
resource_id = resource.aws_instance.dev.id
64-
devolutions_gateway_version = "2025.1.6" # Specify a specific version
60+
devolutions_gateway_version = "2025.2.2" # Specify a specific version
6561
}
6662
```
67-
68-
## Roadmap
69-
70-
- [ ] Test on Microsoft Azure.

registry/coder/modules/windows-rdp/main.test.ts

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@ import {
88

99
type TestVariables = Readonly<{
1010
agent_id: string;
11-
resource_id: string;
1211
share?: string;
1312
admin_username?: string;
1413
admin_password?: string;
@@ -45,13 +44,11 @@ describe("Web RDP", async () => {
4544
await runTerraformInit(import.meta.dir);
4645
testRequiredVariables<TestVariables>(import.meta.dir, {
4746
agent_id: "foo",
48-
resource_id: "bar",
4947
});
5048

5149
it("Has the PowerShell script install Devolutions Gateway", async () => {
5250
const state = await runTerraformApply<TestVariables>(import.meta.dir, {
5351
agent_id: "foo",
54-
resource_id: "bar",
5552
});
5653

5754
const lines = findWindowsRdpScript(state)
@@ -96,7 +93,6 @@ describe("Web RDP", async () => {
9693
import.meta.dir,
9794
{
9895
agent_id: "foo",
99-
resource_id: "bar",
10096
},
10197
);
10298

@@ -116,7 +112,6 @@ describe("Web RDP", async () => {
116112
import.meta.dir,
117113
{
118114
agent_id: "foo",
119-
resource_id: "bar",
120115
admin_username: customAdminUsername,
121116
admin_password: customAdminPassword,
122117
},

registry/coder/modules/windows-rdp/main.tf

Lines changed: 5 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ terraform {
44
required_providers {
55
coder = {
66
source = "coder/coder"
7-
version = ">= 0.17"
7+
version = ">= 2.5"
88
}
99
}
1010
}
@@ -35,11 +35,6 @@ variable "agent_id" {
3535
description = "The ID of a Coder agent."
3636
}
3737

38-
variable "resource_id" {
39-
type = string
40-
description = "The ID of the primary Coder resource (e.g. VM)."
41-
}
42-
4338
variable "admin_username" {
4439
type = string
4540
default = "Administrator"
@@ -53,14 +48,14 @@ variable "admin_password" {
5348

5449
variable "devolutions_gateway_version" {
5550
type = string
56-
default = "2025.2.1"
51+
default = "2025.2.2"
5752
description = "Version of Devolutions Gateway to install. Defaults to the latest available version."
5853
}
5954

6055
resource "coder_script" "windows-rdp" {
6156
agent_id = var.agent_id
6257
display_name = "windows-rdp"
63-
icon = "/icon/desktop.svg"
58+
icon = "/icon/rdp.svg"
6459

6560
script = templatefile("${path.module}/powershell-installation-script.tftpl", {
6661
admin_username = var.admin_username
@@ -101,7 +96,7 @@ resource "coder_app" "rdp-docs" {
10196
agent_id = var.agent_id
10297
display_name = "Local RDP Docs"
10398
slug = "rdp-docs"
104-
icon = "https://raw.githubusercontent.com/matifali/logos/main/windows.svg"
105-
url = "https://coder.com/docs/ides/remote-desktops#rdp-desktop"
99+
icon = "/icon/windows.svg"
100+
url = "https://coder.com/docs/user-guides/workspace-access/remote-desktops#rdp"
106101
external = true
107102
}

0 commit comments

Comments
 (0)