-
Notifications
You must be signed in to change notification settings - Fork 3
RDP-specific Coder Desktop deep link #52
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
Comments
relates to #52 Adds support for Coder Desktop to handle the `coder:/` URI scheme by registering for this scheme and forwarding activations to the single instance of Coder Desktop. Also removes the `Package.appxmanifest`. It is unused since Coder Desktop is not a packaged app.
Thinking about this a bit more, I think the URI format should be:
We should include the authority, so that if you have multiple Coder deployments and you click a link from one of them, and Coder Desktop is signed into a different one, the you can get a sensible error. We should prefer name references over UUIDs, so that it's possible to bookmark. If we used Agent UUIDs, the link would break as soon as we rebuild. |
This is a good addition. It would help provide better self-troubleshooting. |
Adds basic support for `coder:/` URIs for opening RDP. relates to #52 but I still need to add support for checking the authority.
Add RDP-specific logic into Coder Desktop so that all coder apps marked with
external=true
and a URL matching tocoder:/v0/open/agent/<agent ID>/rdp?username=<username>&password=<passwpord>
will be rendered as buttons and have RDP-specific logic in place. This logic can include fetching additional information or credentials in Coder Desktop and creating an.rdp
file or comparable integrations.The text was updated successfully, but these errors were encountered: