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

RFD 51: X11 forwarding #10009

Merged
merged 5 commits into from
Feb 3, 2022
Merged

RFD 51: X11 forwarding #10009

merged 5 commits into from
Feb 3, 2022

Conversation

Joerger
Copy link
Contributor

@Joerger Joerger commented Jan 28, 2022

Current implementation - #9897

@github-actions github-actions bot requested review from Tener and zmb3 January 28, 2022 02:37
@github-actions github-actions bot added rfd Request for Discussion tsh tsh - Teleport's command line tool for logging into nodes running Teleport. labels Jan 28, 2022
@Joerger Joerger mentioned this pull request Jan 28, 2022
@Joerger Joerger requested a review from jakule January 28, 2022 02:48
@Joerger Joerger force-pushed the rfd/0051-x11-forwarding branch from e1b56dd to 35220e5 Compare January 28, 2022 02:51
Copy link
Collaborator

@zmb3 zmb3 left a comment

Choose a reason for hiding this comment

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

A few minor comments, but LGTM.

rfd/0051-x11-forwarding.md Outdated Show resolved Hide resolved
rfd/0051-x11-forwarding.md Outdated Show resolved Hide resolved
rfd/0051-x11-forwarding.md Outdated Show resolved Hide resolved
rfd/0051-x11-forwarding.md Outdated Show resolved Hide resolved
rfd/0051-x11-forwarding.md Outdated Show resolved Hide resolved
rfd/0051-x11-forwarding.md Outdated Show resolved Hide resolved
@Joerger Joerger force-pushed the rfd/0051-x11-forwarding branch from 69a7198 to c3ff6d5 Compare January 29, 2022 04:09
@Joerger Joerger requested a review from zmb3 January 31, 2022 19:10
@Joerger
Copy link
Contributor Author

Joerger commented Jan 31, 2022

@zmb3 please take another look, I made significant changes to the UX and security sections.

or other sources to discover the reason, and personally decide whether to switch to `trusted`
forwarding or keep their X Server secure.

This does leave us with the unfortunate fact that the UX of `tsh` and `ssh` will be different
Copy link
Collaborator

Choose a reason for hiding this comment

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

If this became a problem, we could technically add some code to detect if tsh is aliased and being invoked as ssh, and supporting the ssh flags only in that case.

Right?

@jakule
Copy link
Contributor

jakule commented Feb 2, 2022

@Joerger Do we have any plans to add audit to X11 sessions? I'm not sure if this even possible, but technically having that enabled allows you to run X11 session and do some stuff that goes "under the radar" of our auditing.

@Joerger
Copy link
Contributor Author

Joerger commented Feb 3, 2022

@Joerger Do we have any plans to add audit to X11 sessions? I'm not sure if this even possible, but technically having that enabled allows you to run X11 session and do some stuff that goes "under the radar" of our auditing.

The X11 session is logged in the audit log, but the X11 session is not currently being recorded, I assume that's what you're referring to. It's true that a user could do something like this, such as something simple like opening vscode and deleting/altering files without it being recorded in the session.

This might be possible but hasn't been requested/discussed yet, and we've decided to skip it for the MVP.

But when/if we do it, we'd basically have to tee the byte stream from the X Server Proxy to a session log. Then that byte stream could be sent to the player's $DISPLAY (We may need to add an XServer to the Web UI ??). This step would probably not be so trivial, but I think it would be possible.

@Joerger Joerger enabled auto-merge (squash) February 3, 2022 17:31
@Joerger Joerger merged commit 3cdc2c5 into master Feb 3, 2022
@Joerger Joerger deleted the rfd/0051-x11-forwarding branch February 3, 2022 17:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rfd Request for Discussion tsh tsh - Teleport's command line tool for logging into nodes running Teleport.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants