Skip to content
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

Desktop access troubleshooting docs update #45461

Closed
wants to merge 8 commits into from

Conversation

oshati
Copy link
Contributor

@oshati oshati commented Aug 14, 2024

Add steps to troubleshoot directory sharing on a Remote desktop session

Reference:
https://github.com/gravitational/customer-sensitive-requests/issues/300#issuecomment-2253150468

Copy link

🤖 Vercel preview here: https://docs-7gofts9vh-goteleport.vercel.app/docs/ver/preview

@oshati oshati added documentation no-changelog Indicates that a PR does not require a changelog entry labels Aug 14, 2024
Copy link

🤖 Vercel preview here: https://docs-j76h63r2n-goteleport.vercel.app/docs/ver/preview

@oshati oshati requested a review from pschisa August 14, 2024 14:55
Copy link

🤖 Vercel preview here: https://docs-kylpkwm0b-goteleport.vercel.app/docs/ver/preview

@oshati oshati marked this pull request as ready for review August 14, 2024 17:05
Comment on lines 418 to 420
- Do not allow COM port redirection
- Do not allow drive redirection
- Do not allow LPT port redirection
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wouldn't expect COM port redirection or LPT port redirection to matter for directory sharing.

In fact, I can disable these and directory sharing still works as expected:

image

As soon as I disable device redirection I do get the original error, so I think you can remove these two.

Comment on lines 398 to 402
Attempts to share directory on a remote desktop session results in the error:

```
Encountered shared directory error 1
```
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What do you think about moving this configuration to the setup guide instead of mentioning this poor error message here in the troubleshooting guide?

We can file a separate issue to improve the error message as well.


Teleport requires device redirection policies to be disabled in RDP policies to allow directory sharing from a remote computer in an RDP session

To fix this issue, review and disable the `Device and resource redirection policy` within the `Remote Desktop session host` path below:
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
To fix this issue, review and disable the `Device and resource redirection policy` within the `Remote Desktop session host` path below:
To fix this issue, review the _Device and resource redirection_ group policy settings at the path below:

Co-authored-by: Zac Bergquist <zac.bergquist@goteleport.com>
Co-authored-by: Zac Bergquist <zac.bergquist@goteleport.com>
Copy link

🤖 Vercel preview here: https://docs-dhjoab3yj-goteleport.vercel.app/docs/ver/preview

Copy link

🤖 Vercel preview here: https://docs-b6kfn47hq-goteleport.vercel.app/docs/ver/preview

Co-authored-by: Zac Bergquist <zac.bergquist@goteleport.com>
Copy link

🤖 Vercel preview here: https://docs-bf7qzenlw-goteleport.vercel.app/docs/ver/preview

Copy link

🤖 Vercel preview here: https://docs-jkwjar3ut-goteleport.vercel.app/docs/ver/preview

@ptgott
Copy link
Contributor

ptgott commented Aug 28, 2024

@oshati Just checking to see if this one is ready for another look. Looks like there's a conflict. Thanks!

@zmb3
Copy link
Collaborator

zmb3 commented Aug 28, 2024

This one's on me to take over

zmb3 added a commit that referenced this pull request Aug 29, 2024
zmb3 added a commit that referenced this pull request Aug 29, 2024
@zmb3 zmb3 closed this Aug 29, 2024
zmb3 added a commit that referenced this pull request Sep 3, 2024
zmb3 added a commit that referenced this pull request Sep 3, 2024
zmb3 added a commit that referenced this pull request Sep 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation no-changelog Indicates that a PR does not require a changelog entry size/sm
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants