-
Notifications
You must be signed in to change notification settings - Fork 47
Add Rustdesk module #266
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Add Rustdesk module #266
Conversation
@matifali can you release the pending asset (150$)? |
This is released automatically once the PR is merged and closed |
No comment! But why I'm being delayed as this PR resolve the issue? |
There is no intentional delay, as Algora releases the bounty once the PR is approved, and merged. This is outlined in the Coder Bounty Issue #213 |
@BenraouaneSoufiane Just checking if there is any update on this? |
Fixed, thank you! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR introduces a new RustDesk module that enables remote desktop access within Coder workspaces. The module automates the installation of RustDesk, sets up a virtual display using Xvfb, and provides connection credentials for remote access.
Key changes:
- Adds a complete RustDesk module with cross-platform package manager support
- Implements automatic password generation and virtual display configuration
- Provides Terraform integration with configurable parameters
Reviewed Changes
Copilot reviewed 4 out of 6 changed files in this pull request and generated 5 comments.
File | Description |
---|---|
registry/BenraouaneSoufiane/modules/rustdesk/run.sh |
Main installation and setup script with package manager detection and RustDesk configuration |
registry/BenraouaneSoufiane/modules/rustdesk/main.tf |
Terraform module definition with variables and Coder resource configuration |
registry/BenraouaneSoufiane/modules/rustdesk/README.md |
Module documentation with usage instructions and configuration examples |
registry/BenraouaneSoufiane/README.md |
Author profile information |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@DevelopmentCats thank you for your suggestions, I've committed the suggested fixes |
Tell users how to use/customize the module like rustdesk version, virtual screen dimentions & the password
@BenraouaneSoufiane Can you run |
Formatted, thank you |
@BenraouaneSoufiane Can you remove the package-lock file you accidentally left in the commit? |
Sorry for that, it were deleted now, thanks |
|
I think it's related with terraform (not found, even it's installed), all things related with prettier are succeed, so I run terraform fmt command manually with the recursive flag of course which format well my main.tf, the same thing with run.sh, I run shfmt -w path/to/run.sh which format the script, you see the differences |
I was thinking it may an issue with run.sh after using external formatter, I just did a quick review using online shell validator cc: @DevelopmentCats |
…dcoded version. Fix loging so that it will actually output to workspace logs.
…fy virtual display setup
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@DevelopmentCats Everything looks good, but still couldn't figure out what are changes requested by you
@DevelopmentCats I got changes requested but i couldn't see them, can you outline them? |
Closes #79
Description
This PR add new module, install minimal desktop environment (xfce), virtual display, ,rustdesk package from deb file, init new screen, export DISPLAY environment variable with last created virtual screen, start new xfce session & execute the rustdesk cli, generate new password, change the default password, then log the ID & password to be used within rustdesk client to connect to the host
Type of Change
Module Information
Overview/test video: live demo that launch rustdesk with GUI in a docker container https://youtu.be/_rR-l7nARN4




Screenshots:
Path:
registry/BenraouaneSoufiane/modules/rustdesk
New version:
v1.0.0
Breaking change: [ ] Yes [x] No
Testing & Validation
bun test
)bun run fmt
)Related Issues
/claim #79 (remain asset 150$)